Simpler way of inheriting rules from multiple schemas

see spec/schema/inheriting_schema_spec.rb ....
Situation : I have a number of fields that are shared throughout my application which are not nested schema - but i would like to define a single-field validation schema for each and include as needed in other schema ...  the schema-inheritance works for nested (compound) hashes - see above.   can we also support for non-nested... here's the above spec with 'flat' field added which fails ...
require 'dry-validation'

RSpec.describe 'Reusing schemas' do
  subject(:schema) do
    Dry::Validation.Schema do
      required(:city).filled

      required(:location).schema(LocationSchema)
      required(:country).schema(CountrySchema)
    end
  end

  before do

    LocationSchema = Dry::Validation.Schema do
      configure { config.input_processor = :form }

      required(:lat).filled(:float?)
      required(:lng).filled(:float?)
    end

    # define a 'flat' (scalar) schema

    CountrySchema = Dry::Validation.Schema do
      configure { config.input_processor = :form }
      required(:country).filled(:str?)
    end

  end

  after do
    Object.send(:remove_const, :LocationSchema)
  end

  it 're-uses existing schema' do
    expect(schema.(city: 'NYC', country: 'usa', location: { lat: 1.23, lng: 45.6 })).to be_success

    expect(schema.(city: 'NYC', country: 'usa', location: { lat: nil, lng: '45.6' }).messages).to eql(
      location: {
        lat: ['must be filled'],
        lng: ['must be a float']
      }
    )
  end
end

Thanks !

I think @selectport is asking for the same thing I want: namely, the ability to reuse schemas outside of a nested field. I'll add the (obvious) suggestion / request that this follow the language already established like so
address_schema = Dry::Validation.Schema do
    required(:city).filled(min_size?: 3)
    required(:street).filled
    required(:code).filled
end

complete_schema = Dry::Validation.Schema do
    required(:email_address, Types::Email).value(:str?)
    optional(:email_label, Types::String).maybe(:str?)
    schema(address_schema)                                     # <-- reusing schema
    required(:name).filled
end

The above would be equivalent to:
complete_schema = Dry::Validation.Schema do
    required(:email_address, Types::Email).value(:str?)
    optional(:email_label, Types::String).maybe(:str?)
    required(:city).filled(min_size?: 3)
    required(:street).filled
    required(:code).filled
    required(:name).filled
end

An additional nicity (though it's not super important to me) would be the ability to override individual fields in a reused schema, like so:
complete_schema = Dry::Validation.Schema do
    required(:email_address, Types::Email).value(:str?)
    optional(:email_label, Types::String).maybe(:str?)
    schema(address_schema) do                           # <-- reusing schema
        required(:street).filled(min_size?: 3, :str?)   # <-- overriding the `:street` field
    end
    required(:name).filled
end

Thanks!!!


v1.11 backports 2022-02-17

not included due to conflicts #18696 -- test: Restructure k8sT/Services.go (@brb)
#18762 -- Alibabacloud fixes (@jaffcheng)
#18790 -- iptables: Fix race condition on ipset removal (@pchaigno)

Once this PR is merged, you can update the PR labels via:
$ for pr in 18762 18790; do contrib/backporting/set-labels.py $pr done 1.11; done

or with
$ make add-label branch=v1.11 issues=18762,18790

/test-backport-1.11

/ci-aks-1.11

/test-1.21-5.4

/test-1.23-4.9

k8s-1.23-kernel-4.9 hit #13071, safe to ignore. It's the only failure, merging.


ACCUMULO-626 Iterator fuzzer/tester

Some early code for automatically testing iterators.

There may be value in running some of these test against an iterator stack and not just a single iterator.

For iterators that can be run at major compaction, how can we exercise delete keys?

For iterators that can be run at major compaction, how can we exercise delete keys?

Very good question! The completely mocked-out IteratorEnvironment is a sign that I haven't thought past the scan case. I'm not even sure if the ColumnFiltering iterator with SortedMapIterator is even a 100% valid in-memory reproduction of how it works in a TabletServer.
I had left input entirely up to the user, so I'm not sure of a simple way to test deletions without potentially messing up some expected table structure. There are some tricky edge cases at compaction that I haven't yet considered how to encapsulate. Combiners on non-full compactions are another good example.

Rebased on top of master and addressed the current comments:

Removed Reflections dependency
Reuse Key/Value in the test iterator stack
New deepCopy test that iterates over multiple copies.
More accurate re-seek test case.

Only outstanding thing I want to do before an initial commit is to run all of the iterators in {{o.a.a.c.iterators}} through this framework in accumulo-test (start the dog-fooding).
Keith had some suggestions about fancier testing (iterator stacks, compaction/scopes, and delete keys), which I'm not sure I want to try to solve right away. I feel like I've let these changes sit for far too long already. I'd be happy to wait if anyone else is interested in working on it short-term, though.

Drat, another thing I just thought of: this is meant to be user facing, and I think it should eventually hit public API. It might be premature to put it there immediately (since this hasn't been vetted by anyone but myself), but does anyone have thoughts on a path to "graduate" into public API status?

but does anyone have thoughts on a path to "graduate" into public API status?

Well currently iterators are not declared as public API.

Well currently iterators are not declared as public API.

Ooo, that is a good point (which I often forget). Perhaps I'll just table the idea for now and we can address if/when we do something with iterators.

Okie doke. Pulled in Dylan's test improvements (thanks again for that!). I think the only thing I have left that I want to do is to wire up as many of the existing iterators we provide for users with some tests in the accumulo-test module.
Any concerns, speak up soon. I'll likely merge over the weekend unless I hear objection to not do so. Then, I will file some follow-on JIRA work for the lingering improvements.


The requested URL /asr-html/ was not found on this server

I followed all the steps.
apache2 is correctly installed
after running
/apiai/asr-server/fcgi-nnet3-decoder --fcgi-socket=:8000
m getting the following output on terminal
/apiai/asr-server/fcgi-nnet3-decoder --feature-type=mfcc --mfcc-config=mfcc.conf --frame-subsampling-factor=3 --max-active=2000 --beam=15.0 --lattice-beam=6.0 --acoustic-scale=1.0 --endpoint.silence-phones=1 --endpoint.rule1.min-trailing-silence=0.5 --endpoint.rule2.min-trailing-silence=0.15 --endpoint.rule3.min-trailing-silence=0.1 --fcgi-socket=:8000
LOG (fcgi-nnet3-decoder[5.4.129~1-90363]:Run():FcgiDecodingApp.cc:199) Listening FastCGI data at ":8000"
LOG (fcgi-nnet3-decoder[5.4.129~1-90363]:CompileLooped():nnet-compile-looped.cc:334) Spent 0.0474169 seconds in looped compilation.
and on accessing localhost/asr-html/ from browser
m getting
The requested URL /asr-html/ was not found on this server

Have you solved it?!


dllimport is defined even when glog is built as a static library causing compilation error

I am on Windows 10, Visual Studio 2015. I install glog through the standard procedure of
git clone https://github.com/google/glog/
mkdir build
cd build
cmake -A x64 ..

It generates a solution file. I build the entire solution and build the INSTALL project.
A static library is generated. However, when I am linking glog with my own code I get compilation error similar to #139
error LNK2019: unresolved external symbol "__declspec(dllimport) public: __cdecl google::LogMessage::LogMessage(char const *,int)" (__imp_??0LogMessage@google@@QEAA@PEBDH@Z)

It is treating glog as if it's build as shared dll library instead of a static library. When I back trace and look into logging.h, GOOGLE_GLOG_DLL_DECL is defined as __declspec(dllimport) which is the root cause of the error. I can work around this if I force define GOOGLE_GLOG_DLL_DECL to be nothing by myself. No other external libraries I am using requires me to define a macro myself to get over this problem, so I am assuming glog should have the capability to take care of this itself. Can someone please fix this?

Do you get the error if you use the build directory package config?

Right now it's not clear in what configuration this can happen. GOOGLE_GLOG_DLL_DECL is defined as __declspec(dllimport) only if GOOGLE_GLOG_DLL_DECL has not been previously defined.
In CMakeLists.txt you can check that for static builds GOOGLE_GLOG_DLL_DECL is transitively defined, overriding the definition of GOOGLE_GLOG_DLL_DECL in logging.h.
Under this circumstances, it seems likely that your target does not link to the imported target glog::glog.

I was able to build and install glog without a problem. The problem happens when I am linking glog with my own code. Below is my CMakeLists with glog related section
find_package( GLOG REQUIRED)
include_directories(${GLOG_INCLUDE_DIRS})
target_link_libraries(${PROJECT_NAME} ${GLOG_LIBRARIES})

For some reason if I use find_package in "config" mode, ${GLOG_INCLUDE_DIRS} is empty, resulting in compilation error (is this another potential bug?)
C1083	Cannot open include file: 'glog/logging.h': No such file or directory

so I am forced to write my own FindGLOG.cmake and use find_package in "module" mode, my FindGLOG.cmake is fairly standard like below
set(PFX86 "PROGRAMFILES(X86)")

# Look for the header file.
find_path(GLOG_INCLUDE_DIRS NAMES glog/logging.h HINTS
        $ENV{${PFX86}}/glog/include
        $ENV{${PFX86}}/google-glog/include)

# Look for the library.
find_library(GLOG_LIBRARY NAMES libglog glog HINTS
            $ENV{${PFX86}}/glog/lib
            $ENV{${PFX86}}/google-glog/lib)

set(GLOG_LIBRARIES ${GLOG_LIBRARY})

include(FindPackageHandleStandardArgs)
find_package_handle_standard_args(GLOG DEFAULT_MSG GLOG_LIBRARIES
                                GLOG_INCLUDE_DIRS)

And for the full compilation arguments it's
/GS /TP /W4 /wd"4127" /Zc:wchar_t /I"C:\Program Files (x86)\glog\include" /I"C:\Program Files (x86)\gflags\Include" /Gm- /O2 /Ob2 /sdl /Fd"testlib.dir\Release\vc140.pdb" /Zc:inline /fp:precise /D "WIN32" /D "_WINDOWS" /D "NDEBUG" /D "GFLAGS_IS_A_DLL=0" /D "CMAKE_INTDIR=\"Release\"" /D "testlib_EXPORTS" /D "_WINDLL" /D "_MBCS" /errorReport:prompt /WX /Zc:forScope /GR /Gd /MD /Fa"Release/" /EHsc /nologo /Fo"testlib.dir\Release\" /Fp"testlib.dir\Release\testlib.pch"

glog does not export neither GLOG_INCLUDE_DIRS nor GLOG_LIBRARIES variables. You're using a third party find module. Please refer the documentation in glog's cmake directory on how to use glog.

Wow, target_link_libraries (myapp glog::glog) actually works. I am still confused how it's able to find logging.h without specifying any include_directories though ....

Another thing I would like to add is target_link_libraries (myapp glog::glog) does not work on Ubuntu if glog is installed through sudo apt-get install libgoogle-glog-dev. I had to build glog from source just like I did in windows to make it work

No wonder. CMake support was added after the latest release (i.e., 0.3.4) of glog.


ListCollections API not exposed on Client

Describe the bug
We're missing the listCollections API on the Client class... this is a "bug", as it is available on the Go client: https://github.com/textileio/go-threads/blob/master/api/client/client.go#L398
To Reproduce
Can't list existing collections on a threaddb
Expected behavior
Should be able to list existing collections on a threaddb
Additional context
This is already available on the Go Client: https://github.com/textileio/go-threads/blob/master/api/client/client.go#L398

Hi Textile team, I was just trying to access this listCollection function by directly installing @textile/threads v0.3.13, but I think I'm doing something wrong.  Can I force update just this package and get this fix, or do I need to wait for a new release of @textile/hub before I can use it?

It will take a new release of textile/hub, we should have that available later this week. Sorry for the delay.


add cnosdb addon

add cnosdb addon.
CnosDB is a high-performance, high-compression, and easy-to-use open-source distributed time-series database.
kbcli cluster list
NAME   NAMESPACE   CLUSTER-DEFINITION   VERSION        TERMINATION-POLICY   STATUS    CREATED-TIME
test   default     cnosdb               cnosdb-2.4.0   Delete               Running   Mar 30,2024 21:41 UTC+0800

missing the cnosdb-cluster part


Don't fire onDidEndTerminalShellExecution with exitCode undefined immediately after first onDidChangeTerminalShellIntegration

I think what is causing problems for @anthonykim1 is that calling executeCommand immediately after onDidChangeTerminalShellIntegration would fire multiple times and end up triggering an invalid onDidEndTerminalShellExecution with exitCode=undefined but the execution was set when it shouldn't have been. We should suppress this event if possible.

Can not reproduce this, maybe something was happening in the profile of the terminal that was launched?

Not sure what has changed but I dont get the undefined exitCode like I demoed, but rather 0,1 depending on the actual result of executeCommand, which is great:
Code being used:
        const myTerm = window.createTerminal();
        window.onDidChangeTerminalShellIntegration(async ({ terminal, shellIntegration }) => {
            if (terminal === myTerm) {
                const execution = shellIntegration.executeCommand('echo "Hello world"');

                window.onDidEndTerminalShellExecution((event) => {
                    if (event.execution === execution) {
                        console.log(`Command exited with code ${event.exitCode}`);
                        traceLog(`HERE ${event.exitCode} HERE I AM WITH THE EXIT CODE`);

                    }
                });
            }
        });


ignite new with local relative path to boilerplate is broken

Steps to reproduce:

Try installing a local boilerplate (e.g. ignite new PizzaApp -b ../ignite-my-boilerplate)

I added a few logging statements.
-----------------------------------------------

🔥 igniting app Foo1
{ directory: '/Users/jh/Code/ReactNative/ignite-bowser',
  moduleName: 'ignite-bowser',
  type: 'directory' }
⠋ adding ignite-bowser{ directory: '/Users/jh/Code/ReactNative/ignite-bowser',
  moduleName: 'ignite-bowser',
  type: 'directory' }
✔ using the Infinite Red boilerplate v3 (code name 'Bowser')
✔ added React Native 0.57.8 in 25.44s
⠋ adding /Users/jh/Code/ReactNative/playground/Foo1/ignite-bowser{ moduleName: '/Users/jh/Code/ReactNative/playground/Foo1/ignite-bowser',
  version: null,
  type: 'npm' }
⠹ adding /Users/jh/Code/ReactNative/playground/Foo1/ignite-bowser{ Error: Command failed: yarn add /Users/jh/Code/ReactNative/playground/Foo1/ignite-bowser --dev
warning @storybook/react-native > @storybook/core > autoprefixer ><EMAIL_ADDRESS>Browserslist 2 could fail on reading Browserslist >3.0 config used in other tools.
warning @storybook/react-native > @storybook/core > css-loader > cssnano > autoprefixer ><EMAIL_ADDRESS>Browserslist 2 could fail on reading Browserslist >3.0 config used in other tools.
warning @storybook/react-native > @storybook/core > css-loader > cssnano > postcss-merge-rules ><EMAIL_ADDRESS>Browserslist 2 could fail on reading Browserslist >3.0 config used in other tools.
warning @storybook/react-native > @storybook/core > css-loader > cssnano > postcss-merge-rules > caniuse-api ><EMAIL_ADDRESS>Browserslist 2 could fail on reading Browserslist >3.0 config used in other tools.
error Package "/Users/jh/Code/ReactNative/playground/Foo1/ignite-bowser" refers to a non-existing file '"/Users/jh/Code/ReactNative/playground/Foo1/ignite-bowser"'.

    at ChildProcess.exithandler (child_process.js:294:12)
    at ChildProcess.emit (events.js:188:13)
    at maybeClose (internal/child_process.js:978:16)
    at Socket.stream.socket.on (internal/child_process.js:395:11)
    at Socket.emit (events.js:188:13)
    at Pipe._handle.close (net.js:610:12)
  killed: false,
  code: 1,
  signal: null,
  cmd:
   'yarn add /Users/jh/Code/ReactNative/playground/Foo1/ignite-bowser --dev',
  stderr:
   `warning @storybook/react-native > @storybook/core > autoprefixer ><EMAIL_ADDRESS>Browserslist 2 could fail on reading Browserslist >3.0 config used in other tools.\nwarning @storybook/react-native > @storybook/core > css-loader > cssnano > autoprefixer ><EMAIL_ADDRESS>Browserslist 2 could fail on reading Browserslist >3.0 config used in other tools.\nwarning @storybook/react-native > @storybook/core > css-loader > cssnano > postcss-merge-rules ><EMAIL_ADDRESS>Browserslist 2 could fail on reading Browserslist >3.0 config used in other tools.\nwarning @storybook/react-native > @storybook/core > css-loader > cssnano > postcss-merge-rules > caniuse-api ><EMAIL_ADDRESS>Browserslist 2 could fail on reading Browserslist >3.0 config used in other tools.\nerror Package "/Users/jh/Code/ReactNative/playground/Foo1/ignite-bowser" refers to a non-existing file '"/Users/jh/Code/ReactNative/playground/Foo1/ignite-bowser"'.\n` }
✖ /Users/jh/Code/ReactNative/playground/Foo1/ignite-bowser was not able to be installed. Is it a valid NPM module?
----------
Command failed: yarn add /Users/jh/Code/ReactNative/playground/Foo1/ignite-bowser --dev
warning @storybook/react-native > @storybook/core > autoprefixer ><EMAIL_ADDRESS>Browserslist 2 could fail on reading Browserslist >3.0 config used in other tools.
warning @storybook/react-native > @storybook/core > css-loader > cssnano > autoprefixer ><EMAIL_ADDRESS>Browserslist 2 could fail on reading Browserslist >3.0 config used in other tools.
warning @storybook/react-native > @storybook/core > css-loader > cssnano > postcss-merge-rules ><EMAIL_ADDRESS>Browserslist 2 could fail on reading Browserslist >3.0 config used in other tools.
warning @storybook/react-native > @storybook/core > css-loader > cssnano > postcss-merge-rules > caniuse-api ><EMAIL_ADDRESS>Browserslist 2 could fail on reading Browserslist >3.0 config used in other tools.
error Package "/Users/jh/Code/ReactNative/playground/Foo1/ignite-bowser" refers to a non-existing file '"/Users/jh/Code/ReactNative/playground/Foo1/ignite-bowser"'.

It seems like a return of #985, but the fix is still in place. I'm puzzled.
Workaround is to use absolute paths ... again.

@jamonholmgren relative and absolute paths are not working on Windows when using Git Bash.
Trying to figure out a workaround, but if I find a definelly solution for the issue I'll send a PR.
For know, the evidence...

Relative paths to boilerplates appear to be fixed with Ignite 3. Still need to investigate windows but I believe we don't currently support an absolute path like C:/path/to/boilerplate since we're matching on ~, ., \\, or /.

I'm going to close this, but if anyone wants to fix the absolute paths on Windows, feel free!


Restore name field in Files Share dialog

@david-bell-hcl Can you approve this pull request? Marcus Felder checked the fix yesterday on lcauto and said it works.

@david-bell-hcl  - can you pls merge this?


Error:Execution failed for task ':app:preDebugAndroidTestBuild'. > Conflict with dependency 'com.android.support:support-annotations' in project ':app'. Resolved versions for app (26.1.0) and test app (27.1.1) differ. See https://d.android.com/r/tools/test-apk-dependency-conflicts.html for details.

Welcome to FirebaseUI and thanks for submitting an issue!
Please take a look at open issues, as well as resolved issues, to see if your issue is either already being addressed, or has been solved by someone else.
If not, please feel free to fill in the following info so we can help faster!
Step 1: Are you in the right place?

For issues or feature requests related to the code in this repository file a GitHub issue.
For general technical questions, post a question on StackOverflow tagged appropriately.
For general Firebase discussion, use the firebase-talk google group
For help troubleshooting your application that does not fall under one of the above categories, reach out to the personalized Firebase support channel

Step 2: Describe your environment

Android device: _____
Android OS version: _____
Google Play Services version: _____
Firebase/Play Services SDK version: _____
FirebaseUI version: ____

Step 3: Describe the problem:
Steps to reproduce:

Observed Results:

What happened?  This could be a description, logcat output, etc.

Expected Results:

What did you expect to happen?

Relevant Code:
// TODO(you): code here to reproduce the problem

@royal008 please fill out the issue template when filing an issue.  As I can see from the issue title, this is not a FirebaseUI issue but a normal dependency issue.  If you need help solving it please try StackOverflow.

I solved this problem by changing in app.gradle as following -
Replacing
implementation 'com.android.support:appcompat-v7:26.0.1'
wihth
implementation 'com.android.support:appcompat-v7:27.1.1'

Replace
androidTestImplementation 'com.android.support.test.espresso:espresso-core:3.0.2'
with
androidTestImplementation 'com.android.support.test.espresso:espresso-core:3.0.1'

implementation 'com.android.support:appcompat-v7:26.0.1'
with
implementation 'com.android.support:appcompat-v7:27.1.

This solution is working only for one project when i create new project same issue is occuring
i know i can replace these line in every project but i  want to know why it is happenning

implementation 'com.android.support:appcompat-v7:26.0.1'
with
implementation 'com.android.support:appcompat-v7:27.1.
This solution is working only for one project when i create new project same issue is occuring i know i can replace these line in every project but i want to know why it is happening.
This happens because while creating project you are not setting correct compileSdkVersion, minSdkVersion & targetSdkVersion. Make sure these three values matches the required version.
@SubhamKuswa

thank you i will try it..
On Tue, May 29, 2018, 12:12 PM abhilashasurmount<EMAIL_ADDRESS>wrote:

implementation 'com.android.support:appcompat-v7:26.0.1'
with
implementation 'com.android.support:appcompat-v7:27.1.
This solution is working only for one project when i create new project
same issue is occuring i know i can replace these line in every project but
i want to know why it is happening.
This happens because while creating project you are not setting correct
compileSdkVersion, minSdkVersion & targetSdkVersion. Make sure
these three values matches the required version.
@SubhamKuswa https://github.com/SubhamKuswa
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/firebase/FirebaseUI-Android/issues/1272#issuecomment-392668914,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Al4ebDAJOsSKndEobXrzt1zXPQ9gtymEks5t3O3BgaJpZM4Tp5BP
.

@SubhamKuswa did you able to solve this problem ? because i am facing the same problem


tendies.ts: daily.adjclose and .adjopen are getting incorrect data

Today at 3 PM CDT, SlickBot posted this:

However, if you go to the website that you're pulling the data from, it shows ROK only lost 0.80%.

Checking out: https://www.tiingo.com/rok/prices you can see that the data for Aug 19 shows a drop of 232.00 down to 228.68, a drop of -1.4%.
Therefore, the 3 PM CDT post pulled old data.

Already fixed in #38 . The new numbers are not available until 5:30 PM Eastern according to the docs. adjclose and adjopen are the numbers adjust for things like stock splits.

At 4:31 PM today, SlickBot posted this:

This is still incorrect.

Accepting PRs

From my quick research, it looks like percent should be calculated off the previous close price; not the open price. I'll fix it tonight.


Crash at evaluating skips evaluation at resume

When the evaluation fails with an uncaught exception the resuming will happen after the exception, it would be more convenient and consistent if resuming would immediately trigger evaluation.

Implementing this is a little bit messy, and perhaps not so useful either, if we want we can automatically trigger a validation by having a flag --start-with-validation which could be combined with the automatic validation which now automatically happens when initializing.


Two-way control is not working

Hello, thank you for your wonderful component!
I'm trying to make a carousel with thumbs, but it seems that controls doesn't work properly. Could you please see demo? Thank you in advance.
https://codesandbox.io/s/74mz4jz646

Hi @IndigoHollow , ref is no longer used cause ReactIdSwiper now is function. Sorry for not update demo page. You can refer working version here

Thank you for the answer. Now thumbs are related with the main Swiper. But it works really strange: click on the last thumb updates the first Swiper and click on the middle thumb do nothing at all. Could you please tell me which way to dig to? Really need to solve the problem.


Display Invoice Rune

This commit adds an invoice rune to the ConnectWallet component.  Most values in that screen come from the backend method /shared/connectwallet and the type is found at app-config.type.ts/WalletConnect.
An invoice rune is added to WalletConnect because that is what is shown on the ConnectWallet.tsx component.  ApplicationActions.SET_WALLET_CONNECT appends an invoice rune if one is found.  If one is not found, ConnectWallet.tsx has a useEffect that checks for a missing invoice rune, and makes a createrune rpc call and then refreshes the component.

Hi @evansmj, Thank you for the PR.
Here are my inputs after reviewing it:

Lint errors: There are two lint errors (use-http.ts Line 114 & ConnectWallet.tsx Line 51). The first one can be removed by adding missing getConnectWallet dependency in the array. For connectwallet missing dependency error,   adding // eslint-disable-next-line react-hooks/exhaustive-deps will disable it for the linter.

New Invoice rune on every refresh: Currently createInvoiceRune is not dependent upon showrunes response; resulting in a new invoice rune creation on every refresh/startup. I ended up with 120 new invoice rune while testing this PR :).

Generating a rune without user action: As runes are directly related to node security, it would be better to generate it with user's active involvement. For that, we can leave the box empty if the rune doesn't exist already with Add button. And generate the rune only if user clicks on the button (image below).

Showrunes: Calling showrunes in the frontend will expose all runes from the node to the browser. I would avoid calling it in the backend also as we are already using another file (env COMMANDO_CONFIG) to get the master rune from. We can save INVOICE_RUNE along with LIGHTNING_PUBKEY and LIGHTNING_RUNE in the same file.

Invoice rune persistance: Currently, newly generate invoice rune only persists in the frontend. It would be better if we can keep the state persistent in backend & frontend both.

In summary, invoice rune user story should look like:
1: On application start, read the INVOICE_RUNE value from COMMANDO_CONFIG.
2: Set the value in backend's CONNECT_WALLET_SETTINGS either by step 1 or set to empty string if missing.
3: Show invoice rune value on the connect wallet screen. Show Add button with tooltip if it is empty otherwise simple copy button.
4: On Add button click, call create rune from the frontend.
5: Once the rune value in received in the backend, save that in the COMMANDO_CONFIG.
6: Also set it in the backend (APP_CONSTANTS?, CONNECT_WALLET_SETTINGS?, lightning.service.ts's constructor(), etc.).
7: Send the whole connectWallet response back to the UI and update the UI with Invoice rune value.

Hey @ShahanaFarooqui im looking through figma and the existing app but i can't find a nice + icon svg.  Is there one you have in mind?  otherwise i have this one:

It looks fine but I would recommend to tag Basak on figma asking for plus icon svg. It will save us some time and reworking effort.

It looks fine but I would recommend to tag Basak on figma asking for plus icon svg. It will save us some time and reworking effort.

Sure i've added the request here
https://www.figma.com/design/09CUihPE1P6CDf7KlqTExu?node-id=203-2197#818167883

hey @ShahanaFarooqui ready for review.  I added a loading state and disable the invoice rune buttons while loading.  And I show toasts for successfully creating an invoice rune and if an error occurs.

Thanks everyone. Looking forward to integrating CLN soon!


Issue with sphinxneeds directives during latexpdf build

latexpdf build leads to an exception if I use .. needextract:: is used. The same happens also if .. needtable:: is used
The html build works fine .....
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/sphinx/events.py", line 94, in emit
    results.append(listener.handler(self.app, *args))
  File "/usr/local/lib/python3.10/site-packages/sphinx_needs/directives/need.py", line 394, in process_need_nodes
    print_need_nodes(app, doctree, fromdocname, found_needs_nodes)
  File "/usr/local/lib/python3.10/site-packages/sphinx_needs/directives/need.py", line 423, in print_need_nodes
    build_need(layout, node_need, app, fromdocname=fromdocname)
  File "/usr/local/lib/python3.10/site-packages/sphinx_needs/layout.py", line 157, in build_need
    new_need_node = lh.get_need_table()
  File "/usr/local/lib/python3.10/site-packages/sphinx_needs/layout.py", line 317, in get_need_table
    func["func"](**func["configs"])
  File "/usr/local/lib/python3.10/site-packages/sphinx_needs/layout.py", line 1081, in _grid_simple
    head_entry += self.get_section("head")
  File "/usr/local/lib/python3.10/site-packages/sphinx_needs/layout.py", line 339, in get_section
    line_ready = self._func_replace(line_parsed)
  File "/usr/local/lib/python3.10/site-packages/sphinx_needs/layout.py", line 438, in _func_replace
    result = func(*func_args, **func_kargs)
  File "/usr/local/lib/python3.10/site-packages/sphinx_needs/layout.py", line 563, in meta_id
    id_ref = make_refnode(
  File "/usr/local/lib/python3.10/site-packages/sphinx/util/nodes.py", line 561, in make_refnode
    node['refuri'] = (builder.get_relative_uri(fromdocname, todocname) +
  File "/usr/local/lib/python3.10/site-packages/sphinx/builders/latex/__init__.py", line 138, in get_relative_uri
    return self.get_target_uri(to, typ)
  File "/usr/local/lib/python3.10/site-packages/sphinx/builders/latex/__init__.py", line 132, in get_target_uri
    raise NoUri(docname, typ)
sphinx.errors.NoUri: ('xxxxxx', None)

The above exception was the direct cause of the following exception:
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/sphinx/cmd/build.py", line 281, in build_main
    app.build(args.force_all, args.filenames)
  File "/usr/local/lib/python3.10/site-packages/sphinx/application.py", line 347, in build
    self.builder.build_update()
  File "/usr/local/lib/python3.10/site-packages/sphinx/builders/__init__.py", line 307, in build_update
    self.build(['__all__'], to_build)
  File "/usr/local/lib/python3.10/site-packages/sphinx/builders/__init__.py", line 376, in build
    self.write(docnames, list(updated_docnames), method)
  File "/usr/local/lib/python3.10/site-packages/sphinx/builders/latex/__init__.py", line 285, in write
    doctree = self.assemble_doctree(
  File "/usr/local/lib/python3.10/site-packages/sphinx/builders/latex/__init__.py", line 349, in assemble_doctree
    self.env.resolve_references(largetree, indexfile, self)
  File "/usr/local/lib/python3.10/site-packages/sphinx_monkeypatch/__init__.py", line 173, in _resolve_references
    self.apply_post_transforms(doctree, fromdocname)
  File "/usr/local/lib/python3.10/site-packages/sphinx/environment/__init__.py", line 642, in apply_post_transforms
    self.events.emit('doctree-resolved', doctree, docname)
  File "/usr/local/lib/python3.10/site-packages/sphinx/events.py", line 94, in emit
    results.append(listener.handler(self.app, *args))
  File "/usr/local/lib/python3.10/site-packages/sphinx_needs/needs.py", line 417, in process_caller
    check_func(app, doctree, fromdocname, current_nodes[check_node])
  File "/usr/local/lib/python3.10/site-packages/sphinx_needs/directives/needextract.py", line 126, in process_needextract
    need_extract = create_need(
  File "/usr/local/lib/python3.10/site-packages/sphinx_needs/layout.py", line 80, in create_need
    env.resolve_references(node_container, docname, env.app.builder)
  File "/usr/local/lib/python3.10/site-packages/sphinx_monkeypatch/__init__.py", line 173, in _resolve_references
    self.apply_post_transforms(doctree, fromdocname)
  File "/usr/local/lib/python3.10/site-packages/sphinx/environment/__init__.py", line 642, in apply_post_transforms
    self.events.emit('doctree-resolved', doctree, docname)
  File "/usr/local/lib/python3.10/site-packages/sphinx/events.py", line 105, in emit
    raise ExtensionError(__("Handler %r for event %r threw an exception") %
sphinx.errors.ExtensionError: Handler <function process_need_nodes at 0x7f09ceace5f0> for event 'doctree-resolved' threw an exception (exception: ('xxxxxxx', None))

Extension error (sphinx_needs.directives.need):
Handler <function process_need_nodes at 0x7f09ceace5f0> for event 'doctree-resolved' threw an exception (exception: ('xxxxxxx', None))
make: *** [Makefile:31: latexpdf] Error 1
 *  The terminal process "/bin/sh '-c', 'make latexpdf'" terminated with exit code: 2.
 *  Terminal will be reused by tasks, press any key to close it.

The .. needtable::  directive works in my case (windows) for various builders latexpdf, html and confluence.
Is it working fine when you build for latex?

@daniel3097
Just an information to the project setup. I have many different files and needs (used for the html build) but, there is only one file out of all files which is additionally build as PDF (via latexpdf). So, what I figured out after your comment is the following:
1.) There is no issue with e.g.: needtable and needextractc for latex as well as latexpdf build, if I have everting in the file which is build via latexpdf. No use of needs defined in other files.
2.) The exception occurs only, if I move the .. tcd:: TCD 1 .... and .. tcd:: TCD 2.... to another file.
Working example located in one rst file:
.. tcd:: TCD 1
   :id: TCD_001
   :tags: requirement

   This is an example ....

.. tcd:: TCD 2
   :id: TCD_002
   :tags: requirement

   This is another example ...

.. needtable::
   :types: tcd
   :style: table
   :columns: id, title, status

.. needextract::
   :filter: id in ['TCD_001', 'TCD_002']

@DirkBodenschatz :
I tried your example and i also faced issues (but it looked different that yours).
Made clean improved a bit - but still errors.
hi
What helped in my case was when i removed :sytle: table in ...needtable::
.. needtable::
:types: req
:columns: id, title, status
.. needextract::
:filter: id in ['R_MIIO-01', 'R_MIIO-02']
i hope this helps in your case as well.
the pdflatex path  looked attractive to me as well to generate a pdf.
but its "tough" - so i started exploring the ebook path (wasn't much better) and the confluence path (works well).
On linux https://github.com/useblocks/sphinx-simplepdf might be worth to try (i have no  experience, but it looks impressive)


is FakeSwitch state readable?

So I have an idea. I don't know exactly how to run with it, but thought this might be a good starting point.
Currently, HomeKit doesn't support multiple occupancy for Location triggers.
I can use a Location Trigger to set off a scene when I leave the house
My wife can do the same
but we can't use a location trigger to only set off when both of us have left.
The way this can be done is to use a Location Trigger on each iPhone to change the state or variable of something, and then use that value to manage the multi occupancy.
For Example:
When Adam Leaves the house, set AdamHome to OFF
When Mrs. Leaves the house, set MrsHome to OFF
When Adam Arrives Home, set AdamHome to ON
When MRs. Arrives Home, Set MrsHome to ON
When AdamHome = OFF and MrsHome = OFF , switch on Roomba!
When AdamHome = ON, Switch Off (Dock) Roomba
When MrsHome = ON, Switch Off (Dock) Roomba
Can multiple accessories for FakeSwitch be installed on one HomeBridge?

Hey! Sorry for the late answer.
Multiple accessories of FakeSwitch can be installed.
The module stores the state of an accessory in the persist folder. The name is the accessory's name combined with _conf.txt

Also, I am planning on creating this idea with an also-open source iPhone mobile app.


Variable "time_reference" in input file is not found

This is not a high priority. I will focus now more on DBS and VAD scans. This issue related to FIXED stare can be fixed later.
The older input files have variable "time_reference" within the Sweep group. In a newer file (different institute, newer lidar model), this variable is outside the Sweep group. That creates the following error when running FIXED stare.
processing 'Windcube-fixed/-stare' setting!
reading file 'C:\Users\dhara\OneDrive\Documents\customers\PROBE\VMG_2023\data\MeteoSwiss\Payerne\2023\202301\20230101\WLS200s-197_2023-01-01_00-01-33_fixed_287_50mTP.nc'
Traceback (most recent call last):
File "C:\Users\dhara\anaconda3\lib\runpy.py", line 197, in _run_module_as_main
return _run_code(code, main_globals, None,
File "C:\Users\dhara\anaconda3\lib\runpy.py", line 87, in run_code
exec(code, run_globals)
File "C:\Users\dhara\anaconda3\Scripts\hpl2netCDF-client.exe_main.py", line 7, in
File "C:\Users\dhara\anaconda3\lib\site-packages\hpl2netCDF_client\command_line.py", line 81, in main
proc_dl.dailylvl1()
File "C:\Users\dhara\anaconda3\lib\site-packages\hpl2netCDF_client\hpl2netCDF_client.py", line 1646, in dailylvl1
nc_name= hpl_files.combine_lvl1(hpl_list, confDict, date_chosen)
File "C:\Users\dhara\anaconda3\lib\site-packages\hpl2netCDF_client\hpl_files\hpl_files.py", line 135, in combine_lvl1
ds = xr.concat((hpl_files.read_wcsradial(iit,confDict) for iit in hpl_list.name)
File "C:\Users\dhara\anaconda3\lib\site-packages\xarray\core\concat.py", line 224, in concat
first_obj, objs = utils.peek_at(objs)
File "C:\Users\dhara\anaconda3\lib\site-packages\xarray\core\utils.py", line 196, in peek_at
peek = next(gen)
File "C:\Users\dhara\anaconda3\lib\site-packages\hpl2netCDF_client\hpl_files\hpl_files.py", line 135, in
ds = xr.concat((hpl_files.read_wcsradial(iit,confDict) for iit in hpl_list.name)
File "C:\Users\dhara\anaconda3\lib\site-packages\hpl2netCDF_client\hpl_files\hpl_files.py", line 903, in read_wcsradial
,{ 'units': "seconds since {}".format(pd.to_datetime(ds_tmp.time_reference.data).strftime('%Y-%m-%d %H:%M:%S'))
File "C:\Users\dhara\anaconda3\lib\site-packages\xarray\core\common.py", line 246, in getattr
raise AttributeError(
AttributeError: 'Dataset' object has no attribute 'time_reference'

This now creates the LVL1 output file. But I get the following error when trying to process LVL2 (which it shouldn't do anyway I guess).
Traceback (most recent call last):
File "C:\Users\dhara\anaconda3\lib\runpy.py", line 197, in _run_module_as_main
return _run_code(code, main_globals, None,
File "C:\Users\dhara\anaconda3\lib\runpy.py", line 87, in run_code
exec(code, run_globals)
File "C:\Users\dhara\anaconda3\Scripts\hpl2netCDF-client.exe_main.py", line 7, in
File "C:\Users\dhara\anaconda3\lib\site-packages\hpl2netCDF_client\command_line.py", line 82, in main
proc_dl.dailylvl2()
File "C:\Users\dhara\anaconda3\lib\site-packages\hpl2netCDF_client\hpl2netCDF_client.py", line 1796, in dailylvl2
ds_lvl2.attrs['title']= confDict['NC_TITLE']
UnboundLocalError: local variable 'ds_lvl2' referenced before assignment

When I try to plot backscatter, I get this error:
no backscatter data in file, plot CNR instead
Traceback (most recent call last):
File "C:\Users\dhara\anaconda3\lib\runpy.py", line 197, in _run_module_as_main
return _run_code(code, main_globals, None,
File "C:\Users\dhara\anaconda3\lib\runpy.py", line 87, in run_code
exec(code, run_globals)
File "C:\Users\dhara\anaconda3\Scripts\hpl2netCDF-client.exe_main.py", line 7, in
File "C:\Users\dhara\anaconda3\lib\site-packages\hpl2netCDF_client\command_line.py", line 86, in main
proc_dl.bckql()
File "C:\Users\dhara\anaconda3\lib\site-packages\hpl2netCDF_client\hpl2netCDF_client.py", line 2071, in bckql
beta_max, time_mean, range_vec, elevation, vmin, vmax = ql_helper(ds, confDict)
File "C:\Users\dhara\anaconda3\lib\site-packages\hpl2netCDF_client\hpl2netCDF_client.py", line 568, in ql_helper
ds['relative_beta'] = ds.cnr
File "C:\Users\dhara\anaconda3\lib\site-packages\xarray\core\common.py", line 246, in getattr
raise AttributeError(
AttributeError: 'Dataset' object has no attribute 'cnr'

Thanks for pointing this out, I created the last problem while dealing with the corrupted netCDF issue. Please check that with this latest fix, no other issues arise. Here is the quicklook output:

Works as expected. Closing this.


Api key not found

Summary
 i have done this following chnanges in Android  manifest , but issue is still present
Reproducible sample code
<MapView
            initialRegion={{
              latitude: 37.78825,
              longitude: -122.4324,
              latitudeDelta: 0.0922,
              longitudeDelta: 0.0421,
            }}
          />

Steps to reproduce

Expected result
i want to get  proper map view functionality
Actual result

React Native Maps Version
1.14.0
What platforms are you seeing the problem on?
Android
React Native Version
0.71.6
What version of Expo are you using?
Not using Expo
Device(s)
samsung a7 android 10
Additional information
No response

@Reeta-7  in which Android manifest you are changing? debug or main?

Main
On Mon, 1 Jul 2024, 1:45 pm Pravinkumar S, @.***> wrote:

@Reeta-7 https://github.com/Reeta-7 in which Android manifest you are
changing? debug or main?
—
Reply to this email directly, view it on GitHub
https://github.com/react-native-maps/react-native-maps/issues/5105#issuecomment-2199521377,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALX3VK4TRU6HQJR6T6RUWHTZKEFZLAVCNFSM6AAAAABKC5Q2OGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOJZGUZDCMZXG4
.
You are receiving this because you were mentioned.Message ID:
@.***>

@Reeta-7 please follow the setup instructions


Add ScopeBox 3.4.3

Monitor and capture everything from DV to 4K with top of the line
waveform and vectorscopes. Built-in ScopeLink allows use of scopes
directly in Premiere Pro, Final Cut Pro X and more.

Dang, requested from the wrong branch. Please ignore.


Slashed zero / type properties

Is your feature request related to a problem? Please describe.
I need to be able to select slashed zero for my numbers in a text style.
Describe the solution you'd like
Typography properties as a token (slashed zero, alternative ligatures etc)

Unfortunately Figma's plugin API does not support this yet.

Closing as this will be added when there is an official token type for this and there is plugin API support for this property


Add support for italics

Hi, this PR adds support for italic text, supported in at least Konsole. Italic mode is enabled similarly to other ones, e.g. bold, but with the control number 3.

I didn't know there were any terminals with italics support! Thanks :+1:

Published as v0.6.3.


Audio Upload not working in production

Uploading images locally to the cloudinary bucket works fine but when uploading from the production environment, a [net::ERR_HTTP2_PING_FAILED] is thrown and the file never gets sent to the server. All other requests work perfectly with no issues.
What could be causing the problem?

Well seems the problem came from the server on which the backend was hosted. It has adjusted and now audio uploads work great.


Date Counter

🛠 Fixes Issue #347

👨💻 Changes proposed

✔ Check List (Check all the applicable boxes)

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

📄 Note to reviewers
📷 Screenshots

@DragonUncaged this extension is continuously asking for event date, even after giving the dates in correct format, kindly check once again

Mine Working Fine


[Error: There was an error sending your trade offer. Please try again later. (8)]

[Error: There was an error sending your trade offer.  Please try again later. (8)]
Every time i'm trying to trade ....
MY TRADE REQUEST EXACTLY IS
{ partnerSteamId: '765611980********',
  itemsFromMe:
   [ { appId: 570,
       contexId: 2,
       amount: 1,
       assetId: '36107323_23013902' },
     { appId: 570,
       contexId: 2,
       amount: 1,
       assetId: '61240980_23013902' } ],
  itemsFromThem: [] }

or
{ partnerSteamId: '765611980********',
  itemsFromMe:
   [ { appid: 570,
       contexid: 2,
       amount: 1,
       assetid: '36107323_23013902' },
     { appid: 570,
       contexid: 2,
       amount: 1,
       assetid: '61240980_23013902' } ],
  itemsFromThem: [] }

any help please ???

contexid -> contextid

i'm really very sorry ... but now it's [Error: 401]
BTW i'm just starting up with this steam thing ... Thank you very much for replying ...

No worries. However this issues tracker is for issues with node-steam-tradeoffers only.
See https://github.com/Alex7Kom/node-steam-tradeoffers/issues/121.
Good luck!

Okay i got it ... My apologies ... Thanks again ...

Ok I've half working code right now ... If i trade to bot account (node steam still running) from my account, bot auto accepts the offer and everything is fine ...
but when i do the same thing from makeOffer it gives me this error
[Error: There was an error sending your trade offer.  Please try again later. (26)]
ex. 1
{ partnerSteamId: '765611980********',
  itemsFromMe:
   [ { appid: 570,
       contextid: 2,
       amount: 1,
       assetid: '136107323_230139020' },
     { appid: 570,
       contextid: 2,
       amount: 1,
       assetid: '161240980_230139020' } ] }

ex. 2
{ partnerSteamId: '765611982********',
  accessToken: '****',
  itemsFromMe: [ { appid: 570, contextid: 2, amount: 1, assetid: '370181842_0' } ],
  itemsFromThem: [] }

You were already pointed to #121.
This is not an issue with node-steam-tradeoffers. Please double check your own code.


[Module Proposal]:  avm-res-automation-automationaccount (tf)

Check for previous/existing GitHub issues/module proposals

[X] I have checked for previous/existing GitHub issues/module proposals.

Check this module doesn't already exist in the module indexes

[X] I have checked for that this module doesn't already exist in the module indexes; or I'm proposing the module to be migrated from CARML/TFVM.

Bicep or Terraform?
Terraform
Module Classification?
Resource Module
Module Name
avm-res-automation-automationaccount
Module Details
Module for Automation Account and child components like runbooks, schedules, shared resources
Do you want to be the owner of this module?
Yes
Module Owner's GitHub Username (handle)
didayal-msft
(Optional) Secondary Module Owner's GitHub Username (handle)
Poven795909

Hi @didayal-msft ,
Thanks for requesting/proposing to be an AVM module owner!
We just want to confirm you agree to the below pages that define what module ownership means:

Team Definitions & RACI
Shared Specification (Bicep & Terraform)
Module Support

Any questions or clarifications needed, let us know!
If you agree, please just reply to this issue with the exact sentence below (as this helps with our automation 👍):
"I CONFIRM I WISH TO OWN THIS AVM MODULE AND UNDERSTAND THE REQUIREMENTS AND DEFINITION OF A MODULE OWNER"
Thanks,
The AVM Core Team
#RR

NOTE:

This Module Proposal issue MUST remain open until the module is fully developed, tested and published to the relevant registry. Do NOT close the issue before the successful publication is confirmed!
Once the module is fully developed, tested and published to the relevant registry, and the Module Proposal issue was closed, it MUST remain closed.

"I CONFIRM I WISH TO OWN THIS AVM MODULE AND UNDERSTAND THE REQUIREMENTS AND DEFINITION OF A MODULE OWNER"

Hi @didayal-msft ,
Thanks for confirming that you wish to own this AVM module and understand the related requirements and responsibilities!
We just want to ask you to double check a few important things and take the next steps before you start the development.
Please use the following values explicitly as provided in the module index page:

For your module:

ModuleName - avm-res-automation-automationaccount
TelemetryIdPrefix - 46d3xgtf.res.automation-automationaccount - to be used in your module's telemetry

For your module's repository:

Repository name: terraform-azurerm-avm-res-automation-automationaccount

Create the GitHub teams defined in the ModuleOwnersGHTeam and ModuleContributorsGHTeam columns and grant them permissions as described here: @azure/avm-res-automation-automationaccount-module-owners-tf,@azure/avm-res-automation-automationaccount-module-contributors-tf

Follow the instructions here and assign the necessary permissions.

Grant the right level of permissions for  the AVM core team and PG teams on your GitHub repo as described here.
You can now start the development of this module! ✅ Happy coding! 🎉
Please respond to this comment and request a review from the AVM core team once your module is ready to be published! Please include a link pointing to your PR if available. 🙏
Any further questions or clarifications needed, let us know!
Thanks,
The AVM Core Team

NOTE:

This Module Proposal issue MUST remain open until the module is fully developed, tested and published to the relevant registry. Do NOT close the issue before the successful publication is confirmed!
Once the module is fully developed, tested and published to the relevant registry, and the Module Proposal issue was closed, it MUST remain closed.

If this requires a new owner/collaborator I'm happy to get involved :)

Requested @didayal-msft to share an update here

Work in progress. @ASHR4  I will reach out to you by end of this week on the areas you can contribute

Work in progress. @ASHR4  I will reach out to you by end of this week on the areas you can contribute

Hi @didayal-msft, any advancements regarding this? Itching to help out!

@ASHR4 I will be submitting (the first draft of what I have so far) to dev branch tomorrow and then will request to add a few components. Thank you for reaching out!

@ASHR4 I will be submitting (the first draft of what I have so far) to dev branch tomorrow and then will request to add a few components. Thank you for reaching out!

Hi @didayal-msft, how did your first draft go?
Looking forward to contributing on this

Hi @didayal-msft,
I'm still available if you require contributors :)


fix quic_transport_parameters TLS extension code point

Codecov Report

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

@@            Coverage Diff             @@
##           master    #1715      +/-   ##
==========================================
+ Coverage   87.33%   87.36%   +0.03%
==========================================
  Files          96       96
  Lines        6259     6259
==========================================
+ Hits         5466     5468       +2
+ Misses        563      561       -2
  Partials      230      230

Impacted Files
Coverage Δ

internal/handshake/tls_extension.go
100% <ø> (ø)
:arrow_up:

session.go
74.02% <0%> (+0.31%)
:arrow_up:

Continue to review full report at Codecov.

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


add thumb to this slider?

hi, I have searched a lot for this type of progress bar! thanks, it's perfect! but is there a way to add a thumb to this progress bar?! and set it exactly to stop on the segment white space?! thank you very much!

Hey @Azinnilchi ! Thanks for the feedback. There is a new version coming soon - refactored and cleaned up. Regarding your question - I will think about it. So the idea is to have a small circle in the beginning of each segment and when the segment starts 'playing' the thumb should start moving accordingly, am I right ?

hey, thank you for your response! yeap! you've got my Idea! I had a solution for this in my mind, my solution is to use a slider above it, and set the track of the slider transparent! and also we can set the progress to jump in every segment! with a thumb! it's a beautiful design!

@Azinnilchi please check develop branch.


Question about query

There is an endpoint mb has for all genres -  all:  /genre/all?limit=<LIMIT>&offset=<OFFSET>
I can get the first set (offset 0) by doing -
const allGenres = await mbApi.lookup("genre", "all");
What I can't figure out is how to include the limit and offset in my query.  I've tried, {limit: n, offset: n} but no go.
TypeError: inc.join is not a function

Hi, we currently don't have a implementation that handles the genre/all endpoint.
What you can do as a workaround is use the following code to use the query parameters:
const genres = await mbApi.restGet("/genre/all",{
    offset:"2"
});

Let me know if that works for you.

@Haschikeks That works, thank you.  I'll have to take a look at source for other functions (i.e. restGet).


Add CodeCov export and testing to build process.

Would be better to bring this functionality in via a separate PR

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@538d0a9). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #108   +/-   ##
=========================================
  Coverage          ?   49.24%
  Complexity        ?      528
=========================================
  Files             ?       80
  Lines             ?     1259
  Branches          ?        0
=========================================
  Hits              ?      620
  Misses            ?      639
  Partials          ?        0

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 538d0a9...913726f. Read the comment docs.


Add command does not print any message when successful execution.

Description:
Execute the bal persist add command. There is no message printed when it is successfully executed.
It should print a message similar to
Integrated the persist client and entity types generation with the package build process.

Next steps:
1. Define your data model in "persist/model.bal".
2. Execute `bal build` for a package build with the persist client generation.

Incorporated the generation of persist client and entity types into the package build process.

Next steps:
1. Define your data model in "persist/model.bal".
2. Run bal build to generate the persist client during package build.


Paginate through all documents

Hi @parkr and thanks for the great plugin.
Is there a way to paginate through all collections, posts etc? For a sort of home page that aggregates all updates across a site.

Hey @ChrisChinchilla, you would have to iterate over the collections themselves:
{% for document in site.collections %}
  {{ document.title }}
{% endfor %}


Disable Top level Context

You mentioned wanting to try this @jwatson  -- can you give it a shot?

@KingOfBrian we've got some test failures, and for some reason (can't find a diff) NSManagedObject+RZVinylRecord.h is missing declarations for rzv_primaryKey and a few others.

This should be ok now @jwatson -- All the tests passed locally and I kicked travis a second time and it was fine.


Chat text not displaying in mattermost beta app

Summary
Chat text is not displaying in any channels.
Environment Information
We are using gitlab authentication.

Device Name: Huawei P10 (also a problem on Huawei P9)
OS Version: EMUI v 5.1 / Android v 7.0
Mattermost App Version: 1.0 (Build 34)
Mattermost Server Version: 3.10.0

Steps to reproduce
Log in and try to view text.
Expected behavior
Chat text should be visible.
Observed behavior
All channels appear empty. See screenshot for example.

Possible fixes
n/a

Build 39 still has the same issue.

Just to bump; experiencing this with my Huawei Honor 8.

Same as #733


Brass 534

Documentation outlining how to create a default PA cluster without profile.

This looks good, though I don't see a need to constrain image retrieval from AWS. If there is a need, the documentation should reference ECR rather than EKS.


How to insert certificate data into a route template or param file

Having this issue with a customer trying to adopt applier. We're having trouble figuring out how to insert the certificate data with format:
-----BEGIN CERTIFICATE-----
keydata
-----END CERTIFICATE-----

Into a parameter file such that it can be processed/applied into a route template. With a secret, this is simpler, because I can base64 and pass it in as raw secret data without line breaks, but since routes don't handle it the same way, we're struggling with it.

I've found something that works. Not sure if its the most ideal solution.
Certfile:
apiVersion: v1
kind: Template
metadata:
  creationTimestamp: null
  name: secure-route
objects:
- apiVersion: v1
  kind: Route
  metadata:
    annotations:
      openshift.io/host.generated: "true"
    creationTimestamp: null
    labels:
      app: nodejs-mongodb-example
      template: nodejs-mongodb-example
    name: nodejs-mongodb-example
  spec:
    host: nodejs-mongodb-example-sbx-esauer.apps.d1.casl.rht-labs.com
    to:
      kind: Service
      name: nodejs-mongodb-example
      weight: 100
    wildcardPolicy: None
    tls:
      termination: edge
      key: |-
        -----BEGIN PRIVATE KEY-----
        ${KEY}
        -----END PRIVATE KEY-----
      certificate: |-
        -----BEGIN CERTIFICATE-----
        ${CERTIFICATE}
        -----END CERTIFICATE-----
      caCertificate: |-
        -----BEGIN CERTIFICATE-----
        ${CA_CERTIFICATE}
        -----END CERTIFICATE-----
parameters:
- name: KEY
- name: CERTIFICATE
- name: CA_CERTIFICATE

Output cert data with no line breaks:
tr -d '\n' < src/openshift-toolkit/image-management/myorg-openjdk18/certs/myorg-rootCA.pem | sed 's/[\-]\{5\}BEGIN CERTIFICATE[\-]\{5\}//' | sed 's/[\-]\{5\}END CERTIFICATE[\-]\{5\}//'

Param file:
KEY=<key data>
CERTIFICATE=<cert data>
CA_CERTIFICATE=<ca data

As a note for future me, this same issue is present when trying to pass the private key in via the params_from_vars on the command line.

@tylerauerbeck we're in the process of moving towards using an operator/controller for handling of certificates, rather than an IaC approach. Basically, you apply an "empty" route with applier, annotated to indicate it is in need of a cert, and the operator comes along and fills in the cert data.
https://github.com/etsauer/cert-operator


strawberry.ext.mypy_plugin Pydantic 2.8 PydanticModelField.to_argument error 'force_typevars_invariant'

Hello!
It seems the Pydantic 2.8 version introduced a breaking change on PydanticModelField.to_argument adding a new argument:
https://github.com/pydantic/pydantic/commit/8333bd59dd3424811f4e73cdd6e9414fd15bb6d7
Describe the Bug
Here's the mypy stack trace
./file.py:...: error: INTERNAL ERROR -- Please try using mypy master on GitHub:
https://mypy.readthedocs.io/en/stable/common_issues.html#using-a-development-mypy-build
Please report a bug at https://github.com/python/mypy/issues
version: 1.10.1
Traceback (most recent call last):
  File "mypy/semanal.py", line 6714, in accept
  File "mypy/nodes.py", line 1142, in accept
  File "mypy/semanal.py", line 1621, in visit_class_def
  File "mypy/semanal.py", line 1706, in analyze_class
  File "mypy/semanal.py", line 1740, in analyze_class_body_common
  File "mypy/semanal.py", line 1811, in apply_class_plugin_hooks
  File "/.../.venv/lib/python3.12/site-packages/strawberry/ext/mypy_plugin.py", line 482, in strawberry_pydantic_class_callback
    f.to_argument(
TypeError: PydanticModelField.to_argument() missing 1 required positional argument: 'force_typevars_invariant'
./file.py:...: : note: use --pdb to drop into pdb

System Information

Operating system: OSX
strawberry-graphql 0.235.2
pydantic 2.8.2
pydantic-core 2.20.1
mypy 1.10.1
mypy-extensions 1.0.0

Additional Context
Here's closely what the source code look like
@strawberry.interface(name="ResourceBase")
class ResourceBaseSchema:
    resource_id: str
    resource_type: ResourceTypeSchema

@strawberry.experimental.pydantic.type(MyModel, name="MyModel")
class MyModelSchema(ResourceBaseSchema):
    provider: strawberry.auto
    url: strawberry.auto
    name: strawberry.auto
    tags: list[str]

Upvote & Fund

We're using Polar.sh so you can upvote and help fund this issue.
We receive the funding once the issue is completed & confirmed by you.
Thank you in advance for helping prioritize & fund our backlog.

This bug is also affecting me! @lindycoder do you have an ETA on https://github.com/strawberry-graphql/strawberry/pull/3562?


Update skolemize.py

add a function, richardize (just a name trying to make it not confused with simplify or __reduce__ ), modified from skolemize.
The function name does not represent any logician Richard, and maybe the NLTK maintenance party can choose a better name.
The new function changes minimum from skolemize, and instead conversion to CNF (which is not equivalent but equisatisfiable to the input FOL expression), richardize(your_expr).equiv(your_expr) will be True.
The motivation of this function is that I did not find simplify  to work in the way I wished, and for a pretty messy expression, e.g.,  those with a lot of consecutive negations, the new function can make it neater.

@cestwc: it looks like you've duplicated skolemize and made some changes to it. We need a well argued case for having (and maintaining) two similar functions. Could we please have more examples of different behaviours of the two functions? Is there a case for having both?


⚠ Bofrost.be has degraded performance

In 0060681, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1003 ms

Resolved: Bofrost.be performance has improved in 75c611a.


ruler: Use notifier Prometheus package when decoupled from config package.

Rationales: Ensure similar logic and usage of powerful relabelling in Thanos Ruler.
Acceptance Criteria

We don't mainain pkg/alert package and instead use notifier from Prometheus codebase.
We don't depend on config package that depends on lots of 3rd party packages (k8s, consul, ec2, etc)

Right now our code mimics notifier.

Is this work still planned, or is full relabeling in Ruler already supported in some other way?

Hi, I believe for now there are the --alert.label-drop= and --label= flags which I'd say cover most of the use cases?
Let us knou if you are blocked by this and those labels does not suffice you and we can look more into it.
Right now I'm not aware of anyone working on this.

We're still in the early stages of figuring out our Thanos alerting setup, so I'm not sure yet if we'll need richer relabeling or not.  I was just curious since it seemed like the PR linked above was pretty much there, it just needed a final 👍.

@bwplotka  Can we remove this note that links to issue in the docs? https://github.com/thanos-io/thanos/blob/master/docs/components/rule.md#ruler-ha If not all is implemented do you think it makes sense to open a new issue about this?

We might need another issue if we need such, so far we did not seen the need ):  Anyway the comment in doc is confusing (:

We require full alert relabeling within Thanos Rule.
To move forward, we would consider a contribution, but would need some assistance/guidance on how to move forward.
Maybe @bwplotka , how would you proceed or what would you suggest as next step?

IIUC, we still want to use notifier package rather than maintaining our own alert package? I will reopen this issue, feel free to comment and close it if it is unnecessary anymore


Add StrictHostKeyChecking=no ssh option

Add StrictHostKeyChecking=no in populate compute_id file task.
In CI we need to skip hostkey finger check for ssh, otherwise task
hangs waiting on prompt for accepting key check.

This change might not be needed once https://github.com/openstack-k8s-operators/data-plane-adoption/pull/487 is merged

This can be closed as we're no longer ssh-ing into the compute nodes?

https://github.com/openstack-k8s-operators/data-plane-adoption/pull/487 merged, closing this one as it is not required any more


Remove internal kafka

Remove the processing of events on the internal kafka instance, now processing events purely from the Aiven instance.

This PR was successfully deployed to production by CircleCI build 197 at 2017-07-17 11:09 BST.


SyntaxError: invalid syntax

Hello with bash experiments/demo_vox.sh I obtained an error:
~/Talking-Face_PC-AVS-main$ bash experiments/demo_vox.sh
Traceback (most recent call last):
File "inference.py", line 4, in
from options.test_options import TestOptions
File "/home/andrea/Talking-Face_PC-AVS-main/options/test_options.py", line 1, in
from .base_options import BaseOptions
File "/home/andrea/Talking-Face_PC-AVS-main/options/base_options.py", line 5, in
from util import util
File "/home/andrea/Talking-Face_PC-AVS-main/util/util.py", line 47
imgs = np.concatenate([imgs, np.zeros((rowPadding, *imgs.shape[1:]), dtype=imgs.dtype)], axis=0)
^
SyntaxError: invalid syntax

I have the same error
I was just following the instructions to run the tool

I got the same error too, is there a solution?


Disable days with onDayCreate

Hello!
I understand this example:
`
onDayCreate: function(dObj, dStr, fp, dayElem) {
    if (Math.random() < 0.15)
        dayElem.innerHTML += "";

    else if (Math.random() > 0.85) {
        dayElem.innerHTML += "";
        // I want here disable this dayElem.
         }
}`

but i want disable days with event busy.
It's posible to do disable day in OnDayCreate??
Thank you for help me.

I resolve this problem.
if (Math.random() < 0.15)
dayElem.className += ' ' + ' greenClass';
else if (Math.random() > 0.85) {
dayElem.className += ' ' + ' flatpickr-day flatpickr-disabled redClass'`
}
Thank you for your great flatpickr.


Expose bookkeeperDiskWeightBasedPlacementEnabled in broker.conf

Motivation
Allow users to enable bookkeeper DiskWeightBasedPlacement in broker.conf
Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (no)
The public API: (no)
The schema: (no)
The default values of configurations: (no)
The wire protocol: (no)
The rest endpoints: (no)
The admin cli options: (no)
Anything that affects deployment: (no)

Documentation

Does this pull request introduce a new feature? (no)

run  Integration Tests

run integration tests

run integration tests

when i doing local testing and set the bookkeeperDiskWeightBasedPlacementEnabled=true found that standalone can't startup, log stop at:

When i test enable diskWeightBasedPlacement locally, standalone can't startup successful, the output log stoped at:

@codelipenghui weight based placement policy requires configuring bookies to use v3 protocol. there are two options:

make changes to managed ledger to allow using v3 protocol. so that you can use both.

or

wait until we release bookkeeper 4.10. 4.10 will support a mixed mode of both v2 and v3. the release of bookkeeper 4.10 is under discussion.

@sijie Thanks, broker start succeed after set bookkeeperUseV2WireProtocol=false in broker.conf

run Integration Tests


Added Filters and Sort fields

Fixes: https://github.com/oVirt/ovirt-web-ui/issues/18

Added Filters and sort to toolbar
Rewrote Toolbar to patternfly-react component

@gregsheremeta @sjd78 @lwrigh @michalskrivanek

Nice!! This is looking great!

@sjd78 @lwrigh added saving state of filters and sort to local storage, please, review

@bond95 Can you add an updated screenshot or a gif of it?

@lwrigh actually nothing was changed in design, but I would like to know your opinion about sort reset, because now it saving sort state to local storage and there is no way to reset it, only change. So is there needed some button for reset or not?

@bond95 For sort since there are only two options currently (alphabetical or reverse alphabetical order) and the VMs are automatically sorted in alphabetical order I do not see a need for a reset button since the user can just toggle the sort button between alphabetical and reverse alphabetical order. The user could always refresh the page if they would like to reset their sort options.

The user could always refresh the page if they would like to reset their sort options.

@lwrigh  No, they can't because sort is saving to local storage too.

@bond95 Would there be any way to change that? It seems like a strange behavior if the user can't refresh the page to reset the sorting.

@lwrigh yes, of course, I can remove it.

@bond95 Great that sounds good to me. I think that would help to improve the behavior of it.

@bond95 Would there be any way to change that? It seems like a strange behavior if the user can't refresh the page to reset the sorting.

@lwrigh Done.
@sjd78  please, review

@bond95 - the PR does need a small rebase

ci build please

ci build please

ci please test

ci build please

Sorting alphabetically by OS is not sorting correctly according to the OS labels on the VMs, probably connected to the second issue with the list of OS.

List of OS is different from the one you choose in VM detail edit or VM create and is not sorted.

List of status is not sorted.

The filter list with status contains one item called 'not_responding' which is not in the same format as other states.

The filter list with OS types containing three Linux and three Other OS, using each of them results into a different filter tag.

Also, you cannot have a filter with more OS types and states at once, but you can have more names at once. Not sure if this is by design or not.
Sorting VMs alphabetically by the OS type is still not correct.

LGTM

@bond95 please change "Asleep" as well, that doesn't correspond to anything...should be called "Suspended"

Hi guys,
are you able to tell me in which version the enhancement will be in?
The current search in 1.6.0 is not really helpful as it is searching only from the beginning of the VM name. From the screen shots it looks like that this fix is supporting better search capabilities.
Best regards
Timmi

Hi guys,
are you able to tell me in which version the enhancement will be in?
The current search in 1.6.0 is not really helpful as it is searching only from the beginning of the VM name. From the screen shots it looks like that this fix is supporting better search capabilities.
Best regards
Timmi

@TimmiORG - This change was merged and released in version 1.6.0.

Hi guys,
are you able to tell me in which version the enhancement will be in?
The current search in 1.6.0 is not really helpful as it is searching only from the beginning of the VM name. From the screen shots it looks like that this fix is supporting better search capabilities.
Best regards
Timmi

@TimmiORG - This change was merged and released in version 1.6.0.

Hi @sjd78,
then I guess this is not working as it should.
From the design document I understood that the search string will checked at any position of the VM name. Currently it is looking only on the beginning of the VM name.
Example search for "int":
This will find the following:
intel
But not:
a0-int
Let me know if I should create a bug ticket for this one.
Best regards
Timmi


When context is larger than BERT_MAX_LEN

In the forward function here: (https://github.com/Microsoft/SDNet/blob/master/Models/Bert/Bert.py#L57)
You divide the tokens to deal with the context window larger than BERT_MAX_LEN. However, if you do this, then in the subsequent loops, the truncated BERT tokens are no longer valid "bertify" tokens, since it does not contain <CLS> as the beginning of the input sentence, will it cause a problem to get correct BERT representations ?

You're right that the second windows does not contain . You can try adding  and see whether it can improve performance.


Issues with GluonTS library (MXNet Error)

I'm running into an error when running the DeepAREstimator from the GluonTS library. I have downloaded all of the necessary packages, and with using the mx.gpu() function, it recognizes that the GPU (Nvidia Cuda 11.2) exists on the system
Note, that I'm able to run the code fine on Colab without having to run specific commands like installing "libquadmath0" or the NCCL library (ver 2.8.4 for cuda 11.2), but when running on a docker image, this seems to be the case. The docker image version is 19.3.12, but it's running through a Kubernetes cluster on Azure. The kernel version is 4.15.0-1096-azure and the kublet version is 1.17.9.
Here's the full error trace:
---------------------------------------------------------------------------
MXNetError                                Traceback (most recent call last)
<ipython-input-10-80cdc1a37827> in <module>
     29       print("\nPatient {} - Amount of Days (Train): {}\n | Amount of Days (Valid): {}\n".format(p_id, train_days, valid_days))
     30
---> 31       train1_output = estimator.train(training_data=training_data, validation_data=validation_data)
     32     #print(agg_metrics)

/usr/local/lib/python3.6/dist-packages/gluonts/mx/model/estimator.py in train(self, training_data, validation_data, num_workers, num_prefetch, shuffle_buffer_length, cache_data, **kwargs)
    205             num_prefetch=num_prefetch,
    206             shuffle_buffer_length=shuffle_buffer_length,
--> 207             cache_data=cache_data,
    208         ).predictor

/usr/local/lib/python3.6/dist-packages/gluonts/mx/model/estimator.py in train_model(self, training_data, validation_data, num_workers, num_prefetch, shuffle_buffer_length, cache_data)
    177             net=training_network,
    178             train_iter=training_data_loader,
--> 179             validation_iter=validation_data_loader,
    180         )
    181

/usr/local/lib/python3.6/dist-packages/gluonts/mx/trainer/_base.py in __call__(self, net, train_iter, validation_iter)
    377                         epoch_no,
    378                         train_iter,
--> 379                         num_batches_to_use=self.num_batches_per_epoch,
    380                     )
    381                     if is_validation_available:

/usr/local/lib/python3.6/dist-packages/gluonts/mx/trainer/_base.py in loop(epoch_no, batch_iter, num_batches_to_use, is_training)
    308                                 batch_size = loss.shape[0]
    309
--> 310                             if not np.isfinite(ndarray.sum(loss).asscalar()):
    311                                 logger.warning(
    312                                     "Batch [%d] of Epoch[%d] gave NaN loss and it will be ignored",

/usr/local/lib/python3.6/dist-packages/mxnet/ndarray/ndarray.py in asscalar(self)
   2583             raise ValueError("The current array is not a scalar")
   2584         if self.ndim == 1:
-> 2585             return self.asnumpy()[0]
   2586         else:
   2587             return self.asnumpy()[()]

/usr/local/lib/python3.6/dist-packages/mxnet/ndarray/ndarray.py in asnumpy(self)
   2564             self.handle,
   2565             data.ctypes.data_as(ctypes.c_void_p),
-> 2566             ctypes.c_size_t(data.size)))
   2567         return data
   2568

/usr/local/lib/python3.6/dist-packages/mxnet/base.py in check_call(ret)
    244     """
    245     if ret != 0:
--> 246         raise get_last_ffi_error()
    247
    248

MXNetError: Traceback (most recent call last):
  File "../include/mshadow/././././cuda/tensor_gpu-inl.cuh", line 129
Name: Check failed: err == cudaSuccess (209 vs. 0) : MapPlanKernel ErrStr:no kernel image is available for execution on the device

To Reproduce
(Please provide minimal example of code snippet that reproduces the error. For existing examples, please provide link.)
In order to reproduce this error,  you should download these packages first on a Jupyter notebook:
!sudo apt install libquadmath0
!sudo apt install libnccl2=2.8.4-1+cuda11.2 --allow-downgrades -y
!pip install mxnet-cu112
!pip install --upgrade gluonts

Then import the mxnet library as well as these libraries under GluonTS:
from gluonts.dataset.common import ListDataset
from gluonts.model.deepar import DeepAREstimator
from gluonts.mx.trainer import Trainer
from gluonts.dataset.util import to_pandas
from gluonts.evaluation import Evaluator
from gluonts.evaluation.backtest import make_evaluation_predictions
from gluonts.support.util import get_download_path
from gluonts.mx.trainer import Trainer
from gluonts.model.predictor import Predictor

Then, you can import a dataset and convert it into a ListDataSet like so:
import pandas as pd
df = pd.util.testing.makeTimeDataFrame(freq='h')
df.drop(columns=['B', 'C', 'D'], inplace=True)
df = df.rename(columns={'A': 'value'})
training_data = ListDataset(
[{"start": train.index[0], "target": train.value}],
freq = "1H"
)

validation_data = ListDataset(
        [{"start": valid.index[0], "target": valid.value}],
        freq = "1H"
    )

Finally, initialize the DeepAREstimator and train it like so (the train function is what produced the error in the first place):
estimator = DeepAREstimator(freq="1D", prediction_length=28, trainer=Trainer(epochs=60, ctx=ctx,
num_batches_per_epoch=1), num_parallel_samples=1)
train1_output = estimator.train(training_data=training_data, validation_data=validation_data)

Thank you for your help in advance.

Hi @jfrank94 thanks for reporting this issue!

Then, you can import a dataset and convert it into a ListDataSet like so:
training_data = ListDataset(
[{"start": train.index[0], "target": train.value}],
freq = "1H"
)

Could you make this self-contained? I’m assuming train here is a pandas Series: could you define it? Using random data (or all zeros) is fine. But it’s easier to debug if the example is self contained

@lostella Thanks for helping. I've updated my original post with a dummy dataset you can use for testing.

@jfrank94
NameError                                 Traceback (most recent call last)
<ipython-input-2-a98a2494cede> in <module>
      4 df = df.rename(columns={'A': 'value'})
      5 training_data = ListDataset(
----> 6 [{"start": train.index[0], "target": train.value}],
      7 freq = "1H"
      8 )

NameError: name 'train' is not defined

@jfrank94 also how did you set ctx?

@jfrank94 could you confirm that the following snippets reproduces the error? (I've run it on CPU and works fine, I still need to test it on GPU)
import mxnet as mx
import pandas as pd

from gluonts.dataset.common import ListDataset
from gluonts.model.deepar import DeepAREstimator
from gluonts.mx.trainer import Trainer

df = pd.util.testing.makeTimeDataFrame(freq='h')
df.drop(columns=['B', 'C', 'D'], inplace=True)
df = df.rename(columns={'A': 'value'})

training_data = ListDataset(
    [{"start": df.index[0], "target": df.value}],
    freq = "1H"
)

validation_data = ListDataset(
    [{"start": df.index[0], "target": df.value}],
    freq = "1H"
)

estimator = DeepAREstimator(
    freq="1D",
    prediction_length=28,
    trainer=Trainer(
        epochs=60,
        ctx=mx.gpu(),
        num_batches_per_epoch=1
    ),
    num_parallel_samples=1
)

train1_output = estimator.train(
    training_data=training_data,
    validation_data=validation_data
)

@lostella Yes, your code looks good to reproduce the error. The error is coming from the train function which is traced back to MXNet.

sure, ok, I'll give notice in case I'm able to reproduce this

@lostella Are there any updates with reproducing the error?

I couldn't reproduce this based on the snippet above, with the latest gluonts using mxnet-cu102 on a p3.2xlarge instance on AWS


[BugFix] Fix bug with logging request headers.

In PR https://github.com/apache/pinot/pull/8907, RequestIdentity is assumed to be of type HttpRequestIdentity. Using GrpcRequesterIdentity will result in error like cannot be cast to class org.apache.pinot.broker.api.HttpRequesterIdentity.
Potential fix is to have a function in RequesterIdentity called getClientIp that is implemented by all subclasses.

@abhs50 would you be able to pick this up?

We may also add getHttpHeaders() to the interface, and by default returns an empty map

Yes I will look into this.

@vvivekiyer : Just to understand the design here, are you suggesting we move https://github.com/apache/pinot/blob/67344859c270b9e64638b6691d456e57a7570403/pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerRequestHandler.java#L671 inside a standalone method called getClientIp in org.apache.pinot.broker.api.HttpRequesterIdentity ?

Another option is we could also do below
if (enableClientIpLogging && requesterIdentity instanceof HttpRequesterIdentity)   on https://github.com/apache/pinot/blob/67344859c270b9e64638b6691d456e57a7570403/pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerRequestHandler.java#L678

IMO, I would avoid using instanceOf to achieve this. Here's the reason for the same.
What I was suggesting is as follows:
Have a method called getClientIp in RequesterIdentity. The default implementation can be to return "unknown"/empty. Allow all the subclasses to have their own implementation for getClientIp. The implementation of getClientIp in HttpRequesterIdentity would borrow the code from here.


slide.hidden

While not a show stopper, slide.hidden = true; does not hide the slide.
--Running PP 2016 on a windows 10 machine.

What version of PptxGenJS are you using?

/* PptxGenJS 2.2.0-20180617 */
('We' should add that to the non-min file.  Even though I am using the min I wen't to the non-min first to find the version.)

Please update to the latest version.

Verified that /* PptxGenJS 2.3.0-beta-20180706 */ fixes the issue


[SPARKTA-507][WEB]: Remove finish step and convert cube creation to last step

:shipit:

Comment accepted. Will treat it ASAP

Squashing and checks in progress.


First pass at BDNYC sources

First pass at addressing #7.
This adds nearly 700 objects from the original BDNYC database and includes the script used to generate them.
Some extra clean up is required as some object names are not consistently formatted (eg, double spaced) or have duplicates (eg, see #12 for TWA 27)

Given that I just fixed #8, I think this PR should be closed and reopened as a PR to the main branch.

Changed the branch for now, but if you implement your tests I will withdraw and rebase onto main before resubmitting.


UI parts overlap

Password and other parts of packets sometimes overlap in qt wallet, they appear OK in browser though

happens when claim list is long


MessageFactory.suggestedActions cannot be serialized

Getting this error when I use suggested actions:
Error "activity.suggestedActions.actions.value must be of type object." occurred in serializing the payload

  at Conversations.<anonymous> (C:\Source\cai-bot\cai-bot-app\node_modules\botbuilder\node_modules\botframework-connector\lib\generated\operations\conversations.js:725:42)
  at Generator.next (<anonymous>)
  at C:\Source\cai-bot\cai-bot-app\node_modules\botbuilder\node_modules\botframework-connector\lib\generated\operations\conversations.js:14:71
  at new Promise (<anonymous>)
  at __awaiter (C:\Source\cai-bot\cai-bot-app\node_modules\botbuilder\node_modules\botframework-connector\lib\generated\operations\conversations.js:10:12)
  at Conversations.replyToActivityWithHttpOperationResponse (C:\Source\cai-bot\cai-bot-app\node_modules\botbuilder\node_modules\botframework-connector\lib\generated\operations\conversations.js:678:16)
  at Conversations.replyToActivity (C:\Source\cai-bot\cai-bot-app\node_modules\botbuilder\node_modules\botframework-connector\lib\generated\operations\conversations.js:1504:25)
  at next (C:\Source\cai-bot\cai-bot-app\node_modules\botbuilder\lib\botFrameworkAdapter.js:430:62)
  at Promise (C:\Source\cai-bot\cai-bot-app\node_modules\botbuilder\lib\botFrameworkAdapter.js:450:13)
  at new Promise (<anonymous>)

Using "botbuilder": "^4.0.0-preview1.2"
I am sending a very simple set of options, copied from docs:
MessageFactory.suggestedActions(['red', 'green', 'blue'], `Choose a color`);

The serialized value is shown in the error text:
{
  "actions": [
    {
      "type": "imBack",
      "value": "red",
      "title": "red"
    },
    {
      "type": "imBack",
      "value": "green",
      "title": "green"
    },
    {
      "type": "imBack",
      "value": "blue",
      "title": "blue"
    }
  ]
}

The CardAction API does specify value property as an object, but it varies depending on the action type.
If I modify cardFactory.ts to completely omit value, it works as expected. I guess the workaround would be to send an array of Action objects rather than strings.

Issue appears to originate in this commit.

For now this workaround fixes the issue:
import { CardAction } from 'botbuilder/node_modules/botframework-connector/lib/generated/models/mappers';

CardAction.type.modelProperties.value.type.name = 'String';

I'll try to reproduce the issue.

Which channel are you using? I created a new bot from scratch and the issue doesn't repo for me using the emulator. Here's my code:
import { BotFrameworkAdapter, MemoryStorage, ConversationState, MessageFactory } from 'botbuilder';
import * as restify from 'restify';

// Create server
let server = restify.createServer();
server.listen(process.env.port || process.env.PORT || 3978, function () {
    console.log(`${server.name} listening to ${server.url}`);
});

// Create adapter
const adapter = new BotFrameworkAdapter({
    appId: process.env.MICROSOFT_APP_ID,
    appPassword: process.env.MICROSOFT_APP_PASSWORD
});

// Define conversation state shape
interface EchoState {
    count: number;
}

// Add conversation state middleware
const conversationState = new ConversationState<EchoState>(new MemoryStorage());
adapter.use(conversationState);

// Listen for incoming requests
server.post('/api/messages', (req, res) => {
    // Route received request to adapter for processing
    adapter.processActivity(req, res, async (context) => {
        if (context.activity.type === 'message') {
            const msg = MessageFactory.suggestedActions(['red', 'green', 'blue'], `Choose a color`);
            await context.sendActivity(msg);
        } else {
            await context.sendActivity(`[${context.activity.type} event detected]`);
        }
    });
});

This is in web chat client. Also, I’m using express, but I don’t think that should make a difference. I’ll try to put together an e2e repro tonight or tomorrow.
Sent from mobile

From: Steven Ickman<EMAIL_ADDRESS>Sent: Wednesday, June 13, 2018 6:32 PM
To: Microsoft/botbuilder-js
Cc: Chris Stone; Author
Subject: Re: [Microsoft/botbuilder-js] MessageFactory.suggestedActions cannot be serialized (#279)
Which channel are you using? I created a new bot from scratch and the issue doesn't repo for me using the emulator. Here's my code:
import { BotFrameworkAdapter, MemoryStorage, ConversationState, MessageFactory } from 'botbuilder';
import * as restify from 'restify';
// Create server
let server = restify.createServer();
server.listen(process.env.port || process.env.PORT || 3978, function () {
console.log(${server.name} listening to ${server.url});
});
// Create adapter
const adapter = new BotFrameworkAdapter({
appId: process.env.MICROSOFT_APP_ID,
appPassword: process.env.MICROSOFT_APP_PASSWORD
});
// Define conversation state shape
interface EchoState {
count: number;
}
// Add conversation state middleware
const conversationState = new ConversationState(new MemoryStorage());
adapter.use(conversationState);
// Listen for incoming requests
server.post('/api/messages', (req, res) => {
// Route received request to adapter for processing
adapter.processActivity(req, res, async (context) => {
if (context.activity.type === 'message') {
const msg = MessageFactory.suggestedActions(['red', 'green', 'blue'], Choose a color);
await context.sendActivity(msg);
} else {
await context.sendActivity([${context.activity.type} event detected]);
}
});
});
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fbotbuilder-js%2Fissues%2F279%23issuecomment-397109122&data=02|01|Christopher.Stone%40microsoft.com|61940faf7cca4c76e4ef08d5d17d8d0d|72f988bf86f141af91ab2d7cd011db47|1|0|636645259540238641&sdata=XvhVKuN7BLGABJaHHv16%2Frku6LddFaxInyrfOsGZ75c%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAC8qe5nNRqHqdA6U9xYbat6MgbdJZgGtks5t8ZL-gaJpZM4Umdxw&data=02|01|Christopher.Stone%40microsoft.com|61940faf7cca4c76e4ef08d5d17d8d0d|72f988bf86f141af91ab2d7cd011db47|1|0|636645259540248649&sdata=ooJa0MnnmVkn5npiyAI40%2B%2Faie8rVv7YfKatnLyjdOs%3D&reserved=0.

The issue is in ms-rest-js. botframework-connector requires<EMAIL_ADDRESS>and at some point my private package manager got stuck on<EMAIL_ADDRESS>which should be fine, given the version semantics.
However, there is a regression in ms-rest-js versions 0.2.6 and 0.2.7 that is causing the error described above. See my repo for repro.
Today the highest version of ms-rest-js that satisfies ^0.2.5 is 0.2.8, which appears to resolve the regression, and explains why you were not able to repro the issue using a fresh project.
Does it make sense to update botframework-connector/package.json to<EMAIL_ADDRESS>

Added PR #289

Closing per review on #289 which was merged into master 2 days ago.


Security scan fails with an empty result list when used from a NuGet package. (OSOE-767)

When imported from a project reference, the ZAP report doesn't fail the test on low risk level or suppressed alerts. The same doesn't seem to be true when used form the NuGet package.
For example check out this run for https://github.com/OrchardCMS/OrchardCore.Commerce/pull/393/commits/b6d2897b68e2401ff8b1e87bfbb7c450d6c081f7. If you download the failure dump, the ZAP report shows no actual alerts so it shouldn't have been created:

On the other hand, if you look through the 2024-01-09-ZAP-Report-localhost.json in the same archive it has warnings for id 10202, which has been suppressed here. We know this suppression works, because that's the reason why the entries don't show up in the HTML report. For now the only workaround I found was to also provide a custom assertion logic that detects and skips the alerts caused by the suppressed code.
Jira issue

The linked code's comment mentions there being a discrepancy between the HTML and JSON reports (the workflow artifacts are expired, unfortunately). So, this seems to be rather a ZAP bug to me than a difference between the source and NuGet versions of the project.

Found this BTW: https://github.com/OrchardCMS/OrchardCore.Commerce/issues/404.

It was an issue in our automation framework plans.


Please  version ios  ipa and xcode 2024

I can't install vega-app on my ipad with xcode I can only install on my android is there a solution or can I install with xcode without having any problem installing with xcode with my ipad pro if there is a solution with an ipa to install easier with an ipa for ios and ipad pro it is easier until the xcode problem is solved to be able to install vegamovie and it is solved soon with xcode  year 2024 agust 18

Building for iOS would be challenging now because the app uses many packages that only work for Android so that code needed to be handled with platform-specific code.


Update examples for 3.0

Updated the examples to reflect the new changes in 3.0
Updated the Edit this page link to point to the correct url in the repo

Let me know if you have any feedback on the wordings I changed, thanks for maintaining this!

Hi @gabriel-flynn thank you very much for your contribution. I will review shortly.


Absolute and relative healthchecks.active.http_path support

The absolute path support permit to define a custom hostname for the active check.
This commit introduce more verbose logs in DEBUG to better track what is checked by the healthcheck (which IP, port, hostname, path).
The use case of this change is to have kong running on multi clusters (not only multi nodes). That permit to fallback to an external service if the local service is unavailable and so to reach behind proxy services
EDIT: please note that a change is necessary on the kong project but not yet submitted as it will depend of the acceptation or not of this pull request.
The kong commit is available at https://github.com/mickybart/kong/commit/77569237ba1c580788a81fd9839e4a9b2e928ce9

Hi hishamhm,
I will try to better describe the needs and what I'm doing.
The global idea is to have X independent kubernetes cluster.
Kong is installed on every clusters with a shared database.
A service can run on all clusters or a subset of clusters.
Each service is configured in the same way (shared database) with 2 targets. The first target is the local service DNS and the second one is a fallback-clouds DNS that is set differently on every clusters.
The fallback-clouds DNS can refer to any other clusters or a subset.
A specific /health is defined for the service with only 1 target: the local one only.
Those targets are members of an upstream object with active check. So by requesting the /health of every targets, nginx will know on which cluster the service is up and running.
With this setup, we need to pass the complete header as the service is not necessary local and need to pass to another proxy (eg: kong node). I forgot to mention that we have a high weight on the local target and 1 on the fallback one + preserve host set to true.
There is multiple ideas behind that:

support multiple cloud providers with one single configuration
support transparent migration between cloud providers
support hybrid deployments with shared uri (some pods on all or subset of clusters).
support healthy/unhealthy without cost of an external cloud provider solution (And we accept the possible extra latency to contact a non local service).

So if a service is not available locally, the client will receive the service from a fallback-clouds healthy.
This is not technically the best solution and there is better than that with avoiding shared uri to have dedicated DNS + external check but this is due to history, money and internal constraints where I'm working on.
Let me know if you need more details on a specific point or if you have a better solution of course.
Thanks for your time

This 5-year-old feature has been missing a CLA agreement since 2021, so I am going to close it. @mickybart, if this is still needed, please reopen.


My wallet

0xDCbE534a50b306055007d32767ece2A94Fc9c2Dd

0xDCbE534a50b306055007d32767ece2A94Fc9c2Dd

0x17427ec1e97102973a713b7e33b8766409f6fc92

0xDCbE534a50b306055007d32767ece2A94Fc9c2Dd

0xDCbE534a50b306055007d32767ece2A94Fc9c2Dd

0x17427ec1e97102973a713b7e33b8766409f6fc92

Uploading asptool_snapshot.txt…

0x6fe06b0C83f5fe2C3684449356cA37719dE3aA80
On Wed, Jul 10, 2024 at 4:05 AM gderq124 @.***> wrote:

0x17427ec1e97102973a713b7e33b8766409f6fc92
—
Reply to this email directly, view it on GitHub
https://github.com/Uniswap/token-lists/issues/684#issuecomment-2219826414,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BBZXPMMISIBZQOBAOA2WG63ZLTTOJAVCNFSM6AAAAABKR2WGXGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMJZHAZDMNBRGQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

0xDCbE534a50b306055007d32767ece2A94Fc9c2Dd

0x17427ec1e97102973a713b7e33b8766409f6fc92

0x17427ec1e97102973a713b7e33b8766409f6fc92

0xDCbE534a50b306055007d32767ece2A94Fc9c2Dd


Identifiers with underscore

I just want to point out that identifiers beginning with underscore and big letter or have double underscores anywhere are reserved in C++ (see http://en.cppreference.com/w/cpp/language/identifiers).
There are multiple identifiers in godot-cpp such as _ArgCast<>, _godot_class_instance_func<>(), ___get_method_class_name<>(), ... .

Huh, didn't know that. I might change that. But maybe not since it's not user facing and if something breaks then I can still patch it without breaking stuff. But thanks for letting me know!

Usually, if one need to use/create some "implementation detail" in header files (as with using templates), detail namespace is used (e.g. namespace godot{ namespace detail { ... }}) for such a thing, no need to use underscore suffixes/prefixes.

Is this still an issue?

Underscores are still present, maybe we could use a godot_internal namespace instead?


query_params.get("param_name") requires a default value

Bug Description
query_params.get("param_name") requires a default value
this default value should be optional and not manadataed
Steps to Reproduce
No response
Your operating system
None
Your Python version (python --version)
None
Your Robyn version
None
Additional Info
No response

Seems valid. I would like to take this up if noone is working on this

https://github.com/sparckles/Robyn/blob/7309086ac1b20c088e87e8028c23abc3f31f3da7/src/types/multimap.rs#L28
The query params are Optional and return None as well