Added MineWebProperties type and getter, using web-properties service.

Added an API method to intermine.api.js to hit the /web-properties service on the mine, which Julie Sullivan so presciently wrote years ago. This PR is against mine-<IP_ADDRESS>-schema of course, but only contains that standalone update which is built on the new layout.

Do we actually want to expose the mine properties in the GraphQL API? I was suggesting we use them internally to make sure the IntermineAPI data source is being used with a compatible mine.

Yeah, I think so, actually. Mine versions can have entirely new objects, which would be entirely unsupported by downstream apps built on an earlier version, and it might be nice for those downstream apps to know that they're on an earlier version than the mine holding the data. But it's totally up to you, you know the servlet URI now and can implement as you wish.
I guess we should store the mine version underlying the GraphQL schema somewhere other than the name of the repo. Seems that should be a const stored in IntermineAPI.

I'm punching this into mine-<IP_ADDRESS>-schema because I wound up with mineWebProperties in it in a spot. Nothing controversial here, we can change it later.


feat: deprecate old SuperChart API that accepts chartProps

💔 Breaking Changes

No longer accept chartProps as a single prop in <SuperChart>
Must specify each field in chartProps individually. e.g.

<SuperChart width={} height={} formData={} />

instead of
<SuperChart chartProps={{width: ... , height: ...,  formData: ...}} />

Please ignore codecov warning. The reduced percentage is due to code deletion.


add logo efihub token

Update the Logo for Efihub Token

please use lower case for file name

please use lower case for file name


Change keeper pdf to contact => request is getting accepted as CONTACT Not KEEPER

Verified this issue on dev app v2.0.01(374)


add stupefacient's corolla tss2 indi tune

Replace birdman's corolla tss2 indi tune for @stupefacient#7567's, and changes the respective UI component.
This tune is quite a bit smoother than birdman's, and also does better in turns.
https://discord.com/channels/880416502577266699/942813754519855104/991067168563154994

Hold on this, there is really significant oscillation with this tune that I didn't realize at first

Change the steer down valves.py 50


Many warnings with node 8.0.0 and npm 5.0

Download new node 8.0.0 for windows. Install. Includes npm 5.0.0
npm install -g create-react-app
create-react-app -V gives version 1.3.1
create-react-app app

Result: many warnings about unmet dependencies + verbose error messages about fsevents.
Fsevents errors were expected on windows but not sure about so many unmet dependencies...

Please show the console output. It’s not enough information to diagnose the issue.

Here are the warnings followed by the error(s).

Also, yarn start seems to work anyways, but I have not tried to go any further, not being confident in this process at this point...

Please update your Yarn version to the latest (0.24.5).

I have added to my original comments on this issue (screenshots).
On Wed, May 31, 2017 at 6:04 PM, Dan Abramov<EMAIL_ADDRESS>wrote:

Please show the console output. It’s not enough information to diagnose
the issue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/facebookincubator/create-react-app/issues/2422#issuecomment-305215427,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABqTyi45XDYu5CD2ybAzhcb7MoIp9EuWks5r_YFrgaJpZM4Nregc
.

The issue is the old buggy Yarn version. If you update Yarn, the issues should go away.
Please let us know if not, and we’ll be happy to reopen!

That's good to know, but by the way I had no yarn before I installed node
and create-react-app, so I think anyone who begins to use create-react-app
since yesterday will encounter the same issue...
Thanks for your insight...
On Wed, May 31, 2017 at 6:18 PM, Dan Abramov<EMAIL_ADDRESS>wrote:

The issue is the old buggy Yarn version. If you update Yarn, the issues
should go away.
Please let us know if not, and we’ll be happy to reopen!
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/facebookincubator/create-react-app/issues/2422#issuecomment-305220292,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABqTylwtAiiw5fYN8GhIFJlJnSVWEhkbks5r_YTAgaJpZM4Nregc
.

I had no yarn before I installed node and create-react-app

The UI definitely looks like Yarn, so maybe it comes preinstalled with whatever way you are using to get Node. Which is fairly unusual. Thanks for reporting, we’ll keep an eye on this.


[Feature Request]: Add Animation for Tab Changes

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
Currently, when changing tabs within the application, there is no animation effect, resulting in a somewhat abrupt transition. Adding a smooth animation during tab changes would enhance the user experience by making the interface feel more responsive and polished.
Use Case
It will enhance user experience.
Benefits
No response
Add ScreenShots
https://github.com/VaibhavCodeClub/learn/assets/98540540/2b0ec861-370d-4d0c-b43a-7f937cc74f22
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I'm a VSoC'24 contributor
[X] I have starred the repository

I would like to work on this enhancement
I am interested in working on this feature and contributing to the project by implementing the proposed solution. Please assign this feature to me so I can start working on it.
Thank you!

@sapatevaibhav , I have made pull request pls, review and merge it.

@sapatevaibhav , I have made pull request pls, review and merge it.


Implement specification validators.

Closes #111. This should be merged after PR #192.
This feature offers the possibility to check if an entity satisfies the conditions in a given specification. Since it's a single entity, it's being validated only against expressions/conditions specified in Where and Search, and everything else is omitted during this process. The usage is quite simple
var customer = new Customer { Name = "something" };
var spec = new CustomerByNameSpec("something");
var result = spec.IsSatisfiedBy(customer)

The implementation uses a separate infrastructure for this feature, by introducing new constructs ISpecificationValidator and IValidator (partial validator). I decided not to use the in-memory evaluators, since we'll force omitted evaluators to implement IsValid(), which is not nice.

Needs docs.


[BUG]: No module named 'colossalai.kernel.extensions'

🐛 Describe the bug
I encountered this problem while running llama pre-training scripts in the application folder, using colossalai0.3.6 installed from source. 'cd ./ColossalAI-0.3.6; BUILD_EXT=1 python -m pip install -e .' .

Environment
colossalai0.3.6
torch2.1.2+cu118
python3.10
gcc10

Correction: I used 'cd ./ColossalAI-0.3.6; BUILD_EXT=1 python -m pip install -e .' ，and encountered the same problem.

Same problem I met. I use Open-Sora to train, stuck on this step.
I notice there exsist a file called 'extensions' in folder ''colossalai/kerner', maybe that's a way to located extensions.py

Same problem I met. I use Open-Sora to train, stuck on this step. I notice there exsist a file called 'extensions' in folder ''colossalai/kerner', maybe that's a way to located extensions.py

This file seems to be a path connected to the outermost “extensions” folder. But pip or python does not seem to recognize it. If you use pip to uninstall colossalai, you may see that it is installed at the same level as colossalai.

Same problem I met. I use Open-Sora to train, stuck on this step. I notice there exsist a file called 'extensions' in folder ''colossalai/kerner', maybe that's a way to located extensions.py

This file seems to be a path connected to the outermost “extensions” folder. But pip or python does not seem to recognize it. If you use pip to uninstall colossalai, you may see that it is installed at the same level as colossalai.

This (https://github.com/hpcaitech/ColossalAI/blob/main/colossalai/kernel/extensions) is a soft link to the parent extension folder. If you pull the main branch you shouldn't encounter this error. Alternatively, you can try `ln -s ../../extensions .'
inside the colossalai.kernel folder.


Upgraded to android 25 / gradle 2.2

Current coverage is 34.82% (diff: 100%)

Merging #116 into master will decrease coverage by 62.50%

@@             master       #116   diff @@
==========================================
  Files            39         39
  Lines          1611       1611
  Methods           0          0
  Messages          0          0
  Branches        280        280
==========================================
- Hits           1568        561   -1007
- Misses           13       1050   +1037
+ Partials         30          0     -30

Powered by Codecov. Last update 9098f74...4aae6b3

LGTM


Provide sample APK

Could you please provide an APK to test this library in case who is lazy :)

Error (404)
We can't find the page you're looking for.


Amend Guide to reflect updated Global Broker metrics

@golfvert proposes updated metrics for the Global Broker in wis2-metric-hierarchy/pull/19 - the problem is that Global Brokers are merging into one metric the absence of metadata and an invalid topic used.
The proposed changes are as follows:

wmo_wis2_gb_messages_no_metadata_total exists as a metric, but it's value is incremented by 1 only if there is no corresponding discovery metadata. Which means that the text in the current Guide [A] would need to be amended to something like:

A Global Broker will check if a discovery metadata record exists corresponding to the topic on which a message has been published. If there is no corresponding discovery metadata record, the Global Broker will discard non-compliant messages and will raise an alert. The metric wmo_wis2_gb_messages_no_metadata_total will be increased by 1. Global Broker should not request GDC for each notification message but should keep a cache of all valid topics for every centre-id.

... The Guide doesn't say how this is validated, but I recall that a GB would maintain a list of topics mentioned in all discovery metadata records, and check that there's a match on that list when validating messages. Does the GB also check that the value of the (optional) properties.metadata_id message attribute, if provided, matches the ID of the discovery metadata record that mentions the topic on which the message has been published?

wmo_wis2_gb_invalid_topic_total is new; a new bullet point should be added to the guide (taking some of the the text in [A]), something like:

A Global Broker will check if the topic on which the message is received is valid. If the topic is invalid, the Global Broker will discard non-compliant messages and will raise an alert. The metric wmo_wis2_gb_invalid_topic_total will be increased by 1.

wmo_wis2_gb_invalid_format_total is new; the text mentioned in [B] should be amended, something like (this text includes the change already committed in the GH version with the correct ref to 1060 Vol II Appendix E):

A Global Broker will validate notification messages against the standard format (see Manual on WIS, Volume II – Appendix E: WIS2 Notification Message), discarding non-compliant messages and raising an alert. The metric wmo_wis2_gb_invalid_format_total will be increased by 1.

wmo_wis2_gb_messages_invalid_total as mentioned in [B] metric is now not used - its function has been split into two separate metrics - see points (2) and (3).

...
The new metrics mentioned in points (2) and (3), and the removal of the metric mentioned in (4) are consistent with the outstanding PR: wis2-metric-hierarchy/pull/19
...
[A]:
From https://library.wmo.int/viewer/68923/download?file=1353_INFCOM-3_en.pdf&type=pdf&navigator=1, #<IP_ADDRESS> bullet 8:

A Global Broker will check if the topic on which the message is received is valid (in particular, a discovery metadata record must exist with a corresponding topic in order that data can be made available using this topic). If the topic is invalid, the Global Broker will discard non-compliant messages and will raise an alert. The metric wmo_wis2_gb_messages_no_metadata_total will be increased by one. Global Broker should not request GDC for each notification message but should keep a cache of all valid topics for every centre-id.

[B]:
From https://library.wmo.int/viewer/68923/download?file=1353_INFCOM-3_en.pdf&type=pdf&navigator=1, #<IP_ADDRESS> bullet 10:

A Global Broker will validate notification messages against the standard format (see Notification message format and structure), discarding non-compliant messages and raising an alert. The metric wmo_wis2_gb_messages_invalid_total will be increased by 1.

Fixed in #164.


Separate utils and logger into separate packages.

I was avoiding this because I felt it was overkill, but it might not be since we are planning on splitting a lot of code up. I might also consider moving some of them into my personal utility package @davecode/utils as i did with asyncMap.

logger is gonna get separated into my own various-web repo. its a simple package and i'll get that done some point soon.
see https://github.com/paperdave/various-web/issues/4


创建kafka DEBEZIUM  数据格式 解析不了数据

Describe the bug
WITH (
connector='kafka',
topic='datawarehouse_dwd_bd_BdStudentclass',
properties.bootstrap.server='<IP_ADDRESS>:9092',
scan.startup.mode='earliest',
scan.startup.timestamp_millis='10000'
) FORMAT DEBEZIUM ENCODE JSON;
Error message/log
No response
To Reproduce
No response
Expected behavior
No response
How did you deploy RisingWave?
No response
The version of RisingWave
No response
Additional context
No response

Could you please provide more information using the bug report issue template?

Please attach some logs from compute nodes. https://docs.risingwave.com/docs/dev/troubleshooting/#review-error-messages-and-logs

Please reopen if have any updates

2023-12-12T06:41:59.771086181Z ERROR actor{otel.name="Actor 1190" actor_id=1190 prev_epoch=5578951995162624 curr_epoch=5578952055324672}:executor{otel.name="DmlExecutor 4A600000001 (actor 1190)" actor_id=1190}:executor{otel.name="FlowControlExecutor (actor 1190)" actor_id=1190}:source_parser{actor_id=1190 source_id=2035}: risingwave_connector::parser: failed to parse message, skipping error=Protocol error: InternalError(TapeError) at character 0 ('S') split_id="0" offset="1564766"


StackRouter warning 'StackRouter provided invalid state, index should always be the top route'

Hi Guys,
I am developing an application that allows the user to move back and forward of the screens in StackNavigation, something like the browser navigation experience. I handled the navigation state myself by extending router.getStateForAction of the StackNavigator.
And I saw a warning StackRouter provided invalid state, index should always be the top route.
Is this a harmless warning? Or Do I miss something here?

It does seem like a harmless warning, but lots of behavior inside of StackRouter.getStateForAction will assume that the current index route is the top route of the stack. So unexpected things may happen in your case because you've provided it with a navigation state like this:
{
  index: 1, // not the last route, as would be expected

Thanks for the feedback. I guess this means that I need to do rigorous tests on my end here and make sure all side-effects are cover. So far it has been working well. I am so happy with the extendability of this library, loving it :)

I am facing the same issue, anyone whom has a solution please share...

If you came from Googling the error: just set your index to your actions array length minus one.
[a, b, c] - set it to 2
[a] - set it to 0
If you need to do something advanced - i'd rather find another library, this one is good for easy stuff


No QRCode

I cloned the sample project but it's not showing any QRcode after running it with npm and also yarn start.

node node_modules/react-native/local-cli/cli.js start
Scanning 575 folders for symlinks in ...
Running packager on port 8081.                                            │
│                                                                            │
│  Keep this packager running while developing on any JS projects. Feel      │
│  free to close this tab and run your own packager instance if you          │
│  prefer.                                                                   │
│                                                                            │
│  https://github.com/facebook/react-native
Looking for JS files in
/Use....
Loading dependency graph...
React packager ready.

I think you're expecting to see a QR code like with Expo / create-react-native-app? This sample was written before that was the standard.. So, there won't be any QR code shown, it's simply "react-native run-android" or "react-native run-ios".
I hope this answers your question?

Thank you for the answer, with the simulator I am getting this error

Have you tried the solution the error message contains? Replacing that piece of code?
If you can, please provide an example of the code triggering this error.

The problem occur when I press "GO TO SETTINGS", this should be the code causing the problem:
Settings: {
    screen: SettingsContainer,
    navigationOptions: {
      title: (navigation) => {
        return navigation.state.params.title
      }
    }
  }

I tried to replace it but it didn't solve the problem.

This seems like it's an issue with react-navigation and not with this project, as I can build and run it just fine. (Please have a look at https://github.com/react-community/react-navigation/issues/1733)
The code you provided is different than the code in the error. This makes me think something else, something react-navigation specific, is triggering the error. Or you adjusted the code which makes the error.
We're going to need more information/the actual code in order to help you.. but might I suggest creating an issue at the react-navigation repository?

You are probably using another version of react-navigation. I think the problem is on the synthax of the code. With this code I do not get the error
Settings: {
    screen: SettingsContainer,
    navigationOptions: (navigation) => ({
      title: 'Login'
    })
  }

Do you have an idea on how to adapt it or shall I open an issue at react-navigation?

This syntax solved the issue for me
Settings: {
  screen: SettingsContainer,
  navigationOptions: ({navigation}) => ({
    title: navigation.state.params.title
  })
}

Okay, I'm glad you got it solved :) Maybe we'll update the project/blogpost to work with the latest versions of dependencies.. but that'll be something for when we have time for that. Pull-requests are encouraged!
Closing.


Unicode headers produce invalid anchors

When redcarpet processes Unicode headers to produce anchors (for use in TOC generation) it returns invalid anchors. The anchor claims to be in UTF-8, but it contains hexadecimal codes.
Worse, the hexadecimal codes are not valid. For example, it returns \xE3\xA9 instead of \xC3\xA9 for an é. I suspect that what is happening is that the string has been incorrectly converted to lowercase, changing \xc3 to \xe3. The lowercase conversion has assumed latin1 encoding when it was actually utf-8. (HT : Janne Karila
Example :
irb(main):001:0> require 'redcarpet'
=> true
irb(main):002:0> markdown = Redcarpet::Markdown.new(Redcarpet::Render::HTML_TOC)
=> #<Redcarpet::Markdown:0x007ff1eb831dc0 @renderer=#<Redcarpet::Render::HTML_TOC:0x007ff1eb831de8>>
irb(main):003:0> test = "# Précis"
=> "# Précis"
irb(main):004:0> rc = markdown.render(test)
=> "
\n
\nPrécis\n
\n
\n"
irb(main):005:0> test.encoding
=> #<Encoding:UTF-8>
irb(main):006:0> rc.encoding
=> #<Encoding:UTF-8>
irb(main):007:0> puts "pr\xC3\xA9cis"
précis
=> nil

https://enterprise.github.com/philoghn

Hi @mitchelltd,
had the same problem today and I sent a pull request with a fix: #528
It was the tolower function which is not locale-aware.
Please tell me if my fix works for you too.

Thank you for reporting and fixing it guys! :-)

Sorry for the delay in replying. This doesn't fix it for me. Nothing has changed in my test example above.
I'm running Mac OS X 10.11.1

Hi @mitchelltd,
that's odd, are you sure you are using the latest master version of redcarpet?
When I try your test with irb and the current master branch of redcarpet, I get:
2.2.3 :001 > require 'redcarpet'
 => true
2.2.3 :002 > markdown = Redcarpet::Markdown.new(Redcarpet::Render::HTML_TOC)
 => #<Redcarpet::Markdown:0x007f7fe940cfc0 @renderer=#<Redcarpet::Render::HTML_TOC:0x007f7fe940cfe8>>
2.2.3 :003 > test = "# Précis"
 => "# Précis"
2.2.3 :004 > rc = markdown.render(test)
 => "
\n
\nPrécis\n
\n
\n"
2.2.3 :005 > test.encoding
 => #<Encoding:UTF-8>
2.2.3 :006 > rc.encoding
 => #<Encoding:UTF-8>
2.2.3 :007 > puts "pr\xC3\xA9cis"
précis
 => nil

Now, the anchor is replaced with a dash.
First, I thought about adding a locale-aware tolower function but this would mean pulling in some iconv or ICU library. (There are so many edge cases, you do not want to code something like this yourself)
So, for my use case it was enough to just replace those non-ASCII characters with -.
Do you need it to be locale-aware lowercase?

Hello @clemensg,
Many thanks ! You are absolutely right. I wasn't using the latest master version. I had checked that I was using the latest master version, but what I was checking was that html.c had changed. But that is insufficient. So I reinstalled with
gem specific_install https://github.com/vmg/redcarpet master
Re locale-aware lowercase. Obviously it would be better to have a locale-aware tolower function. As it is, the code is producing html which is not readable by humans.  Having said that, having anchors that aren't human-readable doesn't actually break anything and should still work.
Thanks again !

Yes I hope we will be more locale-aware in the future but thanks guys ! :-)

Toi chinh sua van de facebook, khi toi lam khac di y ban dau thi khong chap nhan, vi vay toi phai theo dang ky sua lan dau ngay hom nay.Ben nay; ten cua toi; Dao Quang Ngoc, toi muon sua van de dau tien cho khop la; khong co chu VN ghep vao ten chinh cua minh.Cac ban biet hai ten cua toi tai day giong nhau dung khong, va url cua toi chuan dung khong?


TypeScript definitions

add TypeScript definitions (.d.ts file) to the repo.

We should resolve xpl/ansicolor#7 before addressing this.


Execution failed for task ':app:preReleaseBuild'.

FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':app:preReleaseBuild'.

Android dependency 'androidx.localbroadcastmanager:localbroadcastmanager' has different version for the compile (1.0.0-rc01) and runtime (1.0.0) classpath. You should manually set the same version via DependencyResolution

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

########################################
[✓] Flutter (Channel stable, v1.7.8+hotfix.3, on Mac OS X 10.14.5 18F132, locale en-BD)
• Flutter version 1.7.8+hotfix.3 at /Users/tanzimfahim/flutter
• Framework revision b712a172f9 (3 days ago), 2019-07-09 13:14:38 -0700
• Engine revision 54ad777fd2
• Dart version 2.4.0
[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
• Android SDK at /Users/tanzimfahim/Library/Android/sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-28, build-tools 28.0.3
• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 10.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 10.1, Build version 10B61
• CocoaPods version 1.7.4
[✓] iOS tools - develop for iOS devices
• ios-deploy 1.9.4
[✓] Android Studio (version 3.4)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin version 37.0.1
• Dart plugin version 183.6270
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)
[✓] VS Code (version 1.36.1)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.2.0
[✓] Connected device (1 available)
• ONEPLUS A5000 • ceecb6b2 • android-arm64 • Android 9 (API 28)
• No issues found!

I found a solution working for me.
Migrate to androidX and
add the following line in build.gradle (inside app folder)
implementation 'androidx.localbroadcastmanager:localbroadcastmanager:1.0.0'


openAI Proxy  error

Hi Mark, I edited the .example.env file as you said but I get this error and I cannot run the project.

Please check if the OpenAI agent's configuration is correct.
After I re-clone and ran the code, it worked.


Artwork for project

This project needs a logo and a header image. Gelidum means "frozen" in Latin (the language of the Ancient Romans).
Any artist that would like to make a simple logo is greatly appreciated. License of the artwork must be Creative Commons with attribution or Public Domain.
If there is any CC or Public Domain artwork that could match the aim of this project, a comment pointing to it is also appreciated.

A CC-NC image has been added to the README.md file.


Organize bike lane buffer types into a dropdown

#704 added 5 new lane types, for different types of "buffers" between vehicle traffic and bikes. I just jammed in 5 new (poorly drawn) icons into the road editor:

This makes the controls more complicated / choice overload. Since the 5 lane types are logically just variations of one "buffer" type, change it to the persistent split widget (a button + dropdown):

Any opinions @Robinlovelace, @trangrei?

Notes:

if we try to revive the "drag and drop" lane cards, will this complicate it? I don't think so, since this row is for selecting a lane type. To represent the current road's lanes in order, it's still just a bunch of icons.
Say the current buffer type is "planters." Then we click on an existing lane with "stripes." Should we change the persistent split value to be "stripes"?

Previously, all the different lane type controls (traffic, walking, bike, closed with stripes, closed with planter) were more predictable. They all were the same kind of control, they just applied a different flavor of lane.
Now the "stripes" control looks so different, it's hard to guess that it'd behave the same way as the other buttons.
A couple potential solutions:

add a border to the other buttons to make it look more like the dropdown+button thing
use an icon (or icon+text) in the dropdown rather than just text for the closure types
maybe move "construction" from a top level item to an item in the closure dropdown? I'm on the fence about this one though - even though the effect is the same (the lane can't be travelled through) the purpose is different enough that it might be good to keep it top level)

(I'm just brainstorming, I don't feel strongly that my suggestions are the best way forward)

This looks like great progress to me. I'm not 100% sure what has changed and what the options are but both gifs above, and Michael's sketch, look good. The fact that you can even add buffers at all is the main thing for me, a huge step forward, enabling A/B Street to help design more sustainable streets where walking and cycling are the natural choice. Need more time/info and maybe a test before having an opinion on the best option for the dropdown menu and look forward to hearing what others think.


Spring-boot request factory: don't use streaming but buffering implementation

Fixes  #435
Reproducing the issue
The ClientHttpRequestFactories from Spring defaults to using Apache HttpClient 5.x (which we haven't tested our stack against) if it exists on the classpath. This HTTP client has two implementations, a streaming and a buffering one. We picked the wrong one, because we are calling getBody() to access the full content of the response body.
Fix
There is a backwards-compatible implementation which gets picked when you set buffering=true.

👍

The actual validation of the compatibility with Apache HttpClient 5.x will be coming with https://github.com/zalando-nakadi/fahrschein/pull/439.

👍


#157964794 Implement feedback from 3rd checkpoint defence(14/05/2018)

What does this PR do?

Execute the cron job for updating event status every-time the application starts then after every one day instead of just executing it after every one day(This would help to handle issues of server shut-down).
Make use of the success and failure response middleware in the validation codes.
Check the database to validate the role of a user instead of using what was embedded in his/her token(This would ensure you are using the most up-to-date role of that user).
If the user email input is wrong in its format, send a failure response that specifies this instead of sending a response that says Email or password incorrect(This would help the user understand what he/she is doing wrong).
I added onDelete: cascade hook to user-event association.
I also fixed some minor issues such as code documentation, using prope error messages.

Description of Task to be completed?
Implement feedback from defence 14/05/2018
Any background context you want to provide?
None
What are the relevant pivotal tracker stories?
#157964794
Questions:
None

Coverage increased (+0.6%) to 92.853% when pulling 2ba46609d890bd224b4da8952bf3cad01285cda2 on bg-implement-feedback-14/05/2018-157964794 into 3b846a15f928e2b7b1cf217b8bb41a92d39ef6ab on develop.


Cannot pull multiple items of the same type from arsenal

Get Vector21 nite from arsenal, leave arsenal and put vector in backpack
Reenter arsenal get mx-2a, leave arsenal and receive message "following items not allowed: Vector_ace" and have item removed from backpack
Do the same with a weapon and receive the same message

@peakzero
I'll need the arsenal classnames for the items that are getting blacklisted, there is a specific array for stopping that issue but it needs the items.
KP_liberation_config.sqf >
KP_liberation_allowed_items_extension = [     "ACE_Chemlight_Shield_Blue",     "ACE_Chemlight_Shield_Green",     "ACE_Chemlight_Shield_Orange",     "ACE_Chemlight_Shield_Red",     "ACE_Chemlight_Shield_White",     "ACE_Chemlight_Shield_Yellow",     "ACE_launch_NLAW_Used_F",     "ACE_PreloadedMissileDummy_CUP",     "ACE_PreloadedMissileDummy_Igla_CUP",     "ACE_PreloadedMissileDummy_M72A6_CUP",     "ACE_PreloadedMissileDummy_NLAW_CUP",     "ACE_PreloadedMissileDummy_RPG18_CUP",     "ACE_PreloadedMissileDummy_Stinger_CUP",     "ACE_PreloadedMissileDummy_Strela_2_CUP",     ... ...

Adding to v0.96.7a-7Cav-0.2 milestone


Drop get-proxy

It doesn't support NO_PROXY and does not take an input URL (or protocol) to discern which env var to use, as there can be multiple. Meanwhile, proxy-from-env does all of this and more.

I can write a PR for this, if you'd like; taking my tests from https://github.com/stevenvachon/auto-tunnel


Note that CHANGELOG.md (master) is no longer maintained

Per discussion with @cramertj in #993.

As this is only a small addition of a note in CHANGELOG.md, the CI failure is unrelated.

Thanks!


dart2js incorrectly identifies class name as Modifier

dart2js will not generate javascript for the following code fragment:
@interface
abstract class State {

  bool containsKey(final String key);

  State remove(final String key);

  State put(final String key, final Object value);

  Optional<T> get<T>(final String key);

}

Instead, it returns the error:
[Error from Dart2JS on <namespace>|<app name>/index.dart]:

<path>/<filename>.dart:12:3:

Can't have modifier 'Optional' here.

Try removing 'Optional'.

  Optional<T> get<T>(final String key);

  ^^^^^^^^

Whatever a "Modifier" is, dart2js thinks Optional is one :)
The class name is not the problem; renaming Optional to something else still results in the dart2js error, just with a different modifier name.
Could it be something to do with the use of function-level generics?
The code passes strong-mode analysis.
This is using SDK 1.22.1.

Thanks for filing the bug! and Thank you for reducing this to this small repro, it is super helpful and saves us a lot of time when the repros are this small!
The issue seems to be a combination of the generic argument and the fact that the name of the function is get (which is a built-in keyword):
For example: a.dart fails, but b.dart and c.dart are OK:
a.dart:
class State {
  Object get<T>(v) => null;
}
main() => new State();

b.dart:
class State {
  Object getState<T>(v) => null;
}
main() => new State();

c.dart:
class State {
  Object get(v) => null;
}
main() => new State();

Seems like the issue is in the FE parser (see https://github.com/dart-lang/sdk/blob/c78b4d66bb3e0440fdd3020b45f3d00d78082525/pkg/front_end/lib/src/fasta/parser/parser.dart#L1804)

@peter-ahe-google
I notified Peter about a similar issue today, he might be working on it already.

Ah! I see what you mean. I'm perfectly happy to change my method name. Interesting that dart analyzer doesn't flag it as a problem - does that mean it would work in Dartium? It would be good for all the tools to be in agreement.

@ajrcarey you're absolutely right, all tools should be in agreement. They haven't been historically, but we're beginning to share a lot of code moving towards the goal of having a single front end shared by all of our tools.


cacti polkadot connector

Pull Request Requirements

[ ] Rebased onto upstream/main branch and squashed into single commit to help maintainers review it more efficient and to avoid spaghetti git commit graphs that obfuscate which commit did exactly what change, when and, why.
[ ] Have git sign off at the end of commit message to avoid being marked red. You can add -s flag when using git commit command. You may refer to this link for more information.
[ ] Follow the Commit Linting specification. You may refer to this link for more information.

Character Limit

[ ] Pull Request Title and Commit Subject must not exceed 72 characters (including spaces and special characters).
[ ] Commit Message per line must not exceed 80 characters (including spaces and special characters).

A Must Read for Beginners
For rebasing and squashing, here's a must read guide for beginners.

I can review it as it progress

Please also fix the vulnerabilities pointed out by CodeQL


Staging

Motivation for this change
Preparation of next staging merge.
Hope I am not stepping on anybodies toes by proposing this. Just trying to help move things along.
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

Replaced by #55519.


Add  Contributors Images in Readme

@urvashi-code1255 @Vishal-raj-1 Can I work on this issue? Please assign it to me, I would like to solve this as a part of GirlScript Summer of code 2021.

@urvashi-code1255 @Vishal-raj-1 Can I work on this issue? Please assign it to me, I would like to solve this as a part of GirlScript Summer of code 2021.

Sure @Mukta-Sawant go for it :)


add support for crowdbunker.com

Checklist

[X] I'm reporting a new site support request
[X] I've verified that I'm running yt-dlp version 2021.12.27. (update instructions)
[X] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that none of provided URLs violate any copyrights or contain any DRM to the best of my knowledge
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[ ] I've read about sharing account credentials and am willing to share it if required

Region
No response
Example URLs

Single video: https://crowdbunker.com/v/4wETRr7C

Description
« CrowdBunker protects our freedom of expression. It is a platform that is exclusively funded by your donations. Any individual contribution, no matter how small, allows us to expand our capabilities to better protect the freedom of all of us. »
Doesn't require an account to watch videos.
I created a small bash script to download videos from crowdbunker.com that requires curl and ffmpeg
If you want it, don't hesitate to ask me, I don't know where I can make it available. You can use it as a starting point.
Verbose log
> yt-dlp.exe -Uv https://crowdbunker.com/v/4wETRr7C
[debug] Command-line config: ['-Uv', 'https://crowdbunker.com/v/4wETRr7C']
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, err utf-8, pref cp1252
[debug] yt-dlp version 2021.12.27 [6223f67] (win_exe)
[debug] Python version 3.8.10 (CPython 64bit) - Windows-10-10.0.19043-SP0
[debug] exe versions: none
[debug] Optional libraries: Cryptodome, mutagen, sqlite, websockets
[debug] Proxy map: {}
Latest version: 2021.12.27, Current version: 2021.12.27
yt-dlp is up to date (2021.12.27)
[debug] [generic] Extracting URL: https://crowdbunker.com/v/4wETRr7C
[generic] 4wETRr7C: Requesting header
WARNING: [generic] Falling back on generic information extractor.
[generic] 4wETRr7C: Downloading webpage
[generic] 4wETRr7C: Extracting information
[debug] Looking for video embeds
ERROR: Unsupported URL: https://crowdbunker.com/v/4wETRr7C
Traceback (most recent call last):
  File "yt_dlp\YoutubeDL.py", line 1353, in wrapper
  File "yt_dlp\YoutubeDL.py", line 1423, in __extract_info
  File "yt_dlp\extractor\common.py", line 599, in extract
  File "yt_dlp\extractor\generic.py", line 3829, in _real_extract
yt_dlp.utils.UnsupportedError: Unsupported URL: https://crowdbunker.com/v/4wETRr7C

Extractor is really simple to add (I'll make PR soon)
@cayenne17 the api result has these fields
canDelete: false
censored: false
censoredAt: null

Do u have an example of a video where this is true or maybe a hunch as to what a censored video is supposed to mean?
(like it can be accessed only with account or it's not visible but api request will still work or something else???)

Extractor is really simple to add (I'll make PR soon) @cayenne17 the api result has these fields
canDelete: false
censored: false
censoredAt: null

Do u have an example of a video where this is true or maybe a hunch as to what a censored video is supposed to mean? (like it can be accessed only with account or it's not visible but api request will still work or something else???)

Hello,
In fact on Crowdbunker platform, you connect your account of your Youtube channel.
Crowdbunker will upload all the Youtube videos to their servers. When you watch the video on Crowdbunker, if it is not censored on Youtube, then it will watch the video on Youtube. Otherwise, if it is censored from Youtube, then it will watch it from Crowdbunker servers.
Here is an example where it says "CENSORSHIP DETECTED" : https://crowdbunker.com/v/0z4Kms8pi8I

I just found another example "CENSORSHIP DETECTED" :   https://crowdbunker.com/v/8l3mTjK3Po4

@cayenne17 Can u provide a video with subtitles?

subtitles

I found an example where there are subtitles in 3 languages : https://crowdbunker.com/v/85kVlAWA4YM
On the other hand there is an error on this video when it passes by Crowdbunker:
"Impossible de lire la vidéo.
Un problème est survenu lors du chargement de la vidéo.
Error code: hls:networkError_manifestParsingError"

@cayenne17 Can u provide a video with subtitles?

Other examples of censored video with FR subtitles : https://crowdbunker.com/v/wvQlKvLi464

@cayenne17 Can u provide a video with subtitles?

@Ashish0804  Other examples of censored video with FR subtitles : https://crowdbunker.com/v/wvQlKvLi464

Hello,
I just tried to download a Crowdbunker video with the latest version and it doesn't work
> yt-dlp.exe -Uv https://crowdbunker.com/v/4wETRr7C
[debug] Command-line config: ['-Uv', 'https://crowdbunker.com/v/4wETRr7C']
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, err utf-8, pref cp1252
[debug] yt-dlp version 2022.01.21 [f20d607] (win_exe)
[debug] Python version 3.8.10 (CPython 64bit) - Windows-10-10.0.19043-SP0
[debug] exe versions: none
[debug] Optional libraries: Cryptodome, mutagen, sqlite, websockets
[debug] Proxy map: {}
Latest version: 2022.01.21, Current version: 2022.01.21
yt-dlp is up to date (2022.01.21)
[debug] [CrowdBunker] Extracting URL: https://crowdbunker.com/v/4wETRr7C
[CrowdBunker] 4wETRr7C: Downloading JSON metadata
[CrowdBunker] 4wETRr7C: Downloading MPD manifest
[CrowdBunker] 4wETRr7C: Downloading m3u8 information
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), acodec, filesize, fs_approx, tbr, vbr, abr, asr, proto, vext, aext, hasaud, source, id
[debug] Default format spec: best/bestvideo+bestaudio
[info] 4wETRr7C: Downloading 1 format(s): 9
[debug] Invoking downloader on "https://d.video-cdn.divulg.org/cb-medias4/videos/01FS9MANFHCX6GH05FVD8VAT59/</Representation>"
[hlsnative] Downloading m3u8 manifest
ERROR: unable to download video data: HTTP Error 404: Not Found
Traceback (most recent call last):
  File "yt_dlp\YoutubeDL.py", line 3054, in process_info
  File "yt_dlp\YoutubeDL.py", line 2769, in dl
  File "yt_dlp\downloader\common.py", line 440, in download
  File "yt_dlp\downloader\hls.py", line 68, in real_download
  File "yt_dlp\YoutubeDL.py", line 3526, in urlopen
  File "urllib\request.py", line 531, in open
  File "urllib\request.py", line 640, in http_response
  File "urllib\request.py", line 569, in error
  File "urllib\request.py", line 502, in _call_chain
  File "urllib\request.py", line 649, in http_error_default
urllib.error.HTTPError: HTTP Error 404: Not Found


Update drone to prevent new releases being marked as latest

Update the rancher/rancher drone instance to create new releases as pre-releases and also update the docs to add a step to unmark that after the QA validations are done.

Waiting for this PR to be merged: https://github.com/rancher/rancher/pull/44703/files#

https://github.com/rancher/rancher/pull/44758


Updates Nokogiri

It has a security vulnerability and it probably doesn't matter for our purposes but I'm tired of dismissing this warning:

Works for me


Add ability to map ranges from projected C# document to the original …

…document
This is needed to implement a bunch of LSP features in VS Code.

Closing as https://github.com/aspnet/AspNetCore-Tooling/pull/1327 includes this.


Hotfix: Redirect on email confirmation

WHAT DOES THIS PR DO?

Redirect to frontend /login page after email verification

WHAT ISSUES ARE RELATED TO THIS PR?

[X] #88

HOW DO I TEST OUT THIS PR?

N/A

Codecov Report
Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (8702410) 100.00% compared to head (e10a82e) 91.14%.

Files
Patch %
Lines

api/authentication/views.py
28.57%
5 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##              main     #134      +/-   ##
===========================================
- Coverage   100.00%   91.14%   -8.86%
===========================================
  Files            1       28      +27
  Lines           23      666     +643
  Branches         6        0       -6
===========================================
+ Hits            23      607     +584
- Misses           0       59      +59

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


python37Packages.starlette: 0.13.4 -> 0.13.6

Semi-automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/python3.7-starlette/versions.
meta.description for python37Packages.starlette is: "The little ASGI framework that shines"
meta.homepage for python37Packages.starlette is: "https://www.starlette.io/"
meta.changelog for python37Packages.starlette is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.13.6 with grep in /nix/store/v9zvl2vs3zxl2n2lxvxm7fpzm1xkgp0n-python3.7-starlette-0.13.6

directory tree listing: https://gist.github.com/1093472a6ab92ce34d13c29981375ac9

du listing: https://gist.github.com/e47727793e3a79dc7878d6e481f42e36

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

16 total rebuild path(s)

4 package rebuild(s)

4 x86_64-linux rebuild(s)
4 i686-linux rebuild(s)
4 x86_64-darwin rebuild(s)
4 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
python37Packages.fastapi
python37Packages.starlette
python38Packages.fastapi
python38Packages.starlette

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/v9zvl2vs3zxl2n2lxvxm7fpzm1xkgp0n-python3.7-starlette-0.13.6 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/v9zvl2vs3zxl2n2lxvxm7fpzm1xkgp0n-python3.7-starlette-0.13.6
ls -la /nix/store/v9zvl2vs3zxl2n2lxvxm7fpzm1xkgp0n-python3.7-starlette-0.13.6/bin

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

4 packages built:

python37Packages.fastapi
python37Packages.starlette
python38Packages.fastapi
python38Packages.starlette

Maintainer pings
cc @wd15 for testing.

Builds for me.


Add option to preserve diacritics

This also for discussion, and to flesh out what an option to preserve diacritics could look like. (API interface etc not implemented yet, though if this idea has legs I'd be very happy to take that on.)

@dimus here we go; this is updated to preserve diaereses (but not other diacritics) with the -D option. It applies to details, normalized and canonical with the exception of stemmed, which has the diaereses removed (but transliterated directly, i.e. ö -> o, which produces correctly spelled names in my test corpus). Other diacritics (including e.g. ö not preceded by a vowel) are transliterated as they are currently (oe in this example).
I've applied this to the web interface too. The ronn command needs re-running I'm afraid because ronn's dependencies seem to be broken at the moment and I couldn't easily install it.
As always let me know what modifications you need and I'll hop on it. Thanks!

Looks good to me @tobymarsden !  Merging and rebuilding man pages


Exception handling To-do

Currently in project implemented basic handling in windows platform via SEH, but there no filter, destructors and another things that do handlers like C++ or C#
And cause that implemented dumb exception handling via c++ code:
ValueItem* FuncEnviropment::initAndCall(ValueItem* arguments, uint32_t arguments_size) for cleanup and
ValueItem* FuncEnviropment::syncWrapper_catch(ValueItem* args, uint32_t arguments_size) for catching all exceptions, that slow and not flexible for any language
Then i'm think will be good if will implemented fully functional exception handling and created opcodes for handling without proxy or other stuff through native code
Opcodes:

[ ] Handle begin (id)
The beginning of handlng, ids will be unique, or will be thrown InvalidFunction exception

[ ] Handle catch (id, ex count, ex names...)
Where we will jump after catching specific exceptions

[ ] Handle catch (id, value id)
Where we will jump after catching exception specified in value

[ ] Handle catch (id, value id, value meta)
Where we will jump after catching exception specified in value(with meta info)

[ ] Handle finally (id)
Where we will jump after all blocks(jump operations from handle and catch region not jump here, you will implement
that ourselves)
There may be multiple finally blocks

[ ] Handle end(id)
Where handling end

There now unique feature:
"Handle begin" and "Handle end" not define scopes
Without "value hold" all values will be removed at end of function by id order

[ ] Value hold(id, value id)
Define where we need track value

[ ] Value unhold(id)
Define where we need call remove opcode, if from value hold to value unhold thrown exception, then will be called remove operation

"Value hold" need for situations when need safely with exception handling get value, but without moving values, and ability to access value after scope, like var in JavaScript

Issue solved for Windows, but Linux implementation not work


Persistent suggestions in search field

Describe the bug
When using the application's search feature, if the user inputs text and quickly deletes it, the autocomplete suggestions fail to disappear, leaving outdated suggestions visible despite the search field being empty. This issue does not occur if the text is removed slowly, where suggestions disappear as expected. This can confuse users and clutter the interface with irrelevant information.
To Reproduce
Steps to reproduce the behaviour:

Go to the main search page of the application.
Type any query into the search field to trigger autocomplete suggestions.
Rapidly delete all text from the search field.
Notice that the autocomplete suggestions remain visible even though the search field is now empty.

Expected behavior
The expected behavior is that autocomplete suggestions should automatically disappear as soon as the search field is emptied, regardless of how quickly the text is deleted. This ensures that only relevant suggestions are displayed, corresponding to the user's current input.
Screenshots
https://github.com/amosproj/amos2024ss04-building-information-enhancer/assets/46463790/106e81bc-f0fe-46d0-82fd-b832d211a8ce

Desktop (please complete the following information):

OS: macOS Sonoma 14.2.1
Browser: Chrome
Version: 90

Additional context
This bug may be related to the debounce function handling the input field's state or the asynchronous nature of the suggestion-fetching logic, causing a race condition when text is removed quickly.

I think I cannot fix the bug because I have seen that my source has the same behaviour.
Source

Closed as not possible to fix within described scope.


examples: Fix compilation after gio::SimpleAction constructor took …

…the state by value
CC @pbor

It doesn't look like anybody makes sure that gtk3-rs continues compiling :)


pip install not working without cloning repo

Information

qiskit-ibm-provider version: 0.1.0
Python version: 3.8.2
Operating system: Win 10

What is the current behavior?
upon pip install, I get: "ERROR: Could not find a version that satisfies the requirement qiskit-ibm-provider (from versions: none)
ERROR: No matching distribution found for qiskit-ibm-provider"
Steps to reproduce the problem
In an environment where the qiskit-ibm-provider repo has not been cloned, run:
pip install qiskit-ibm-provider
What is the expected behavior?
find and install qiskit-ibm-provider
Suggested solutions

qiskit-ibm-provider has not been released yet that's why you can't directly pip install

ok thx, I was naively following the Readme https://github.com/Qiskit/qiskit-ibm-provider#readme


"Could not decide which case to select" error when using "async"

Some background: I mock many things when writing unit tests for my libraries. In order to make sure I mock things the right way, I assign the type of the original functions to my mocks.
Let's imagine the following:
A model.js file:
// @flow

/*::
type User = {
  id: number,
  email: string
};
*/

const db = require('./db.js');

async function getByIds(ids/*: number*/) {
  if (ids.length === 0) return [];

  const users/*: Array<User>*/ = await db.get({ model: 'user', id: ids });
  return users;
}

exports.getByIds = getByIds;

A service.js file:
// @flow

const model = require('./model.js');

async function getEmails(ids/*: number*/) {
  return (await model.getByIds(ids)).map(x => x.email);
}

exports.getEmails = getEmails;

And the test file test.js:
// @flow

/*::
import * as model from './model.js';
*/

const proxyquire = require('proxyquire');

function runWithAsync() {
  const getByIds/*: typeof model.getByIds*/ = async () => {
    return [];
  };

  const service = proxyquire('./service.js', {
    './model.js': {
      getByIds
    }
  });

  // Test stuff...
}

function runWithoutAsync() {
  const getByIds/*: typeof model.getByIds*/ = () => {
    return Promise.resolve([]);
  };

  const service = proxyquire('./service.js', {
    './model.js': {
      getByIds
    }
  });

  // Test stuff...
}

exports.runWithAsync = runWithAsync;
exports.runWithoutAsync = runWithoutAsync;

And here is the funky thing: even though runWithAsync() and runWithoutAsync() do the same thing, I have the following error for runWithAsync() but no errors for runWithoutAsync():
Error: test.js:11
 11:     return [];
         ^^^^^^^^^^ Promise. Could not decide which case to select
 14: async function getByIds(ids/*: number*/) {
                                             ^ return. See: model.js:14
  Case 1 may work:
   18:   return users;
         ^^^^^^^^^^^^^ type application of async return. See: model.js:18
  But if it doesn't, case 2 looks promising too:
   15:   if (ids.length === 0) return [];
                               ^^^^^^^^^^ type application of async return. See: model.js:15
  Please provide additional annotation(s) to determine whether case 1 works (or consider merging it with case 2):
   11:     return [];
           ^^^^^^^^^^ async return

Found 1 error

When I inspect the type of getByIds() in both runWithAsync() and runWithoutAsync(), they seem to be the same for Flow:
(ids: number) => Promise<[]> | Promise<Array<{email: string, id: number}>>

The type for the original getByIds() function (in model.js) is:
(ids: number) => Promise<[]> | Promise<Array<{email: string, id: number}>>

Another thing I noticed when trying random things: if I declare the mock in model.js using async, there is no problem:
'use strict';

// @flow

/*::
type User = {
  id: number,
  email: string
};
*/

const db = require('./db.js');

async function getByIds(ids/*: number*/) {
  if (ids.length === 0) return [];

  const users/*: Array<User>*/ = await db.get({ model: 'user', id: ids });
  return users;
}

const mockedGetByIds/*: typeof getByIds*/ = async () => {
  return [];
};

exports.getByIds = getByIds;
exports.mockedGetByIds = mockedGetByIds;

Flow is ok with the above. The type computed by Flow for mockedGetByIds is
(ids: number) => Promise<[]> | Promise<Array<{email: string, id: number}>>

Does anyone have an idea why Flow can't make a decision for runWithAsync() but has no problem with runWithoutAsync() and with mockedGetByIds()?

I just ran into another case where union types are not involved this time.
file1.js
// @flow

async function isGreaterThanTen(x/*: number*/) {
  if (x > 10) {
    return true;
  } else {
    return false;
  }
}

exports.isGreaterThanTen = isGreaterThanTen;

file2.js
// @flow

/*::
import * as file1 from './file1.js';
*/

const test/*: typeof file1.isGreaterThanTen*/ = async (x) => {
  return true;
};

Flow outputs:
Error: file2.js:8
  8:   return true;
       ^^^^^^^^^^^^ Promise. Could not decide which case to select
  3: async function isGreaterThanTen(x/*: number*/) {
                                                   ^ return. See: file1.js:3
  Case 1 may work:
    7:     return false;
           ^^^^^^^^^^^^^ type application of async return. See: file1.js:7
  But if it doesn't, case 2 looks promising too:
    5:     return true;
           ^^^^^^^^^^^^ type application of async return. See: file1.js:5
  Please provide additional annotation(s) to determine whether case 1 works (or consider merging it with case 2):
    8:   return true;
         ^^^^^^^^^^^^ async return

Found 1 error

If I rewrite isGreaterThanTen() as below:
async function isGreaterThanTen(x/*: number*/) {
  return x > 10;
}

Flow is happy. Same, if I keep the first version of isGreaterThanTen() and uses Promise.resolve() instead of async in test2.js, Flow is happy:
// @flow

/*::
import * as file1 from './file1.js';
*/

const test/*: typeof file1.isGreaterThanTen*/ = (x) => {
  return Promise.resolve(true);
};

I can't reproduce your problem in flowtry, even though it shows the same types. Are you on the latest version of Flow?
That said, I've had similar problems. When flow says it can't choose between two different cases, you just need to make it explicit:
async function getByIds(ids/*: number*/)/*: Array<User>*/ { // <----
  if (ids.length === 0) return [];

  return await db.get({ model: 'user', id: ids });
}

I can't reproduce your problem in flowtry, even though it shows the same types. Are you on the latest version of Flow?

Yep, as written in the description, the problem doesn't occur when the type of the function is being reused within the same file for some reasons. I guess it's more a matter of context than being in the same file or not. It's probably what is happening when you give it a try in flowtry I think ?

That said, I've had similar problems. When flow says it can't choose between two different cases, you just need to make it explicit:

Thanks 👍
I've run into this issue many times before as well and got around it by explicitly typing the return of my function as you just advised. I also confirm the errors would disappear in this case with that approach.
However, it seems that Flow doesn't know what decision to make when a function re-using another function type uses async, but know what to do when async is not being used. I would like to know if this is a bug and if not a bug, I would like to understand why :)

@stouf have you looked at the inferred type when using await?  I honestly did not read all of this but I have run into an issue with the way $await is defined which causes await to sometimes return the actual Promise class itself, which flow will use in place of objects (and therefore will get confused with unions).
You issue only happens on multiple files - I also can not re-create the issue within a repro using try-flow so it is likely due to using await across files.
try using the following in your example to do refinement:
const whatever = await WhateverElse();
if (!whatever instanceof Promise) {
    //  ... now do you have the issue?
}

@bradennapier Thanks for commenting here!

have you looked at the inferred type when using await?

Yep, I did and the type definitions seem to perfectly match. If we use my second example (way shorter than my first one):
// file2.js

// @flow

/*::
import * as file1 from './file1.js';
*/

const test/*: typeof file1.isGreaterThanTen*/ = async (x) => {
  return true;
};

const a = test(5);

Here, the type of a is Promise<boolean>, which matches with the type of file1.isGreaterThanThen which Flow computes as (x: number) => Promise<boolean>.
However, your assumption about how the Promise class is being computed may be correct. With the same example, if I replace
// file1.js

async function isGreaterThanTen(x/*: number*/) {
  // the code
}

with
async function isGreaterThanTen(x/*: number*/)/*: Promise<*>*/ {
  // the code
}

Flow knows which type to pick up.
I explicitly annotate that the function returns a Promise but I let the existential operator compute the type of the promise.

Any progress?

not enough info to repro


pin jinja2 to fix quart import error

Quart doesn't pin its jinja2 requirement, but it raises an ImportError with the latest minor version. So for now, we'll have to pin jinja2 ourselves.

This was fixed in quart two hours ago, but they haven't made a new release yet.


[FYI] compilation error on gcc13 with tautological-compare

It is almost same issue as https://github.com/cppfastio/fast_io/issues/531.
This is not a bug issue about fast_io, but it may be beneficial to those who use fast_io with gcc13.
If there is no problem, you can close it immediately.
I met a following compilation errors on 20230810 snapshot with gcc 13:
‘constexpr T fast_io::details::compile_time_pow(T, std::size_t) [with T = long unsigned int; std::size_t = long unsigned int]’ used before its definition
  478 | inline constexpr T compile_pow10{::fast_io::details::compile_time_pow<::std::remove_cvref_t<T>>(10,pow)};
      |                                  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~

It seems to be caused by "-Wtautological-compare".
fast_io test code work fine for me with gcc "-Wall -Wextra -Wno-tautological-compare".

-Wall cause this error for my
using fedora 38 with g++ 13.2.1 and CMake 3.27.4

is it fixed?

is it fixed?

Yes, it's working with GCC on Windows and Linux, thank you 😺

looks like a GCC bug

i will file a bug report to GCC to recreate the issue

fixed


Add Sentiment Analysis Example Using Anthropics' Claude Model

Pull Request: Add Sentiment Analysis Example Using Anthropics' Claude Model
Overview
This pull request introduces a new example notebook (SentimentAnalysis.ipynb) to the examples folder. The notebook demonstrates how to perform sentiment analysis using the aisuite library with Anthropics' Claude-3-5-sonnet model.
Key Features

Integration with Anthropics' Claude Model: Uses the aisuite library to interact with Claude-3-5-sonnet.
Sentiment Analysis Workflow: Includes prompts and configurations to analyze text sentiment (positive/negative).
Environment Setup: Demonstrates API key setup using environment variables for secure integration.
Reusable Example: Provides a clear and reusable structure for developers to perform sentiment analysis with LLMs.

File Details
SentimentAnalysis.ipynb

Dependencies

Install the necessary library using:pip install anthropic

API Key Setup

Securely configure API keys using:import os
os.environ["ANTHROPIC_API_KEY"] = "<your-anthropic-api-key>"

Sentiment Analysis Code

Prompt the model to analyze text sentiment (positive/negative).
Example response:Pakistan: The Sentiment is positive

Testing

Verified functionality with real API keys and the Anthropics library.
Output is consistent with the prompts and expectations.

Why This Contribution?
This example provides a practical use case for developers leveraging aisuite for natural language processing tasks. It aligns with the repository's goal of offering unified access to multiple LLM providers.
Checklist

[x] Added the SentimentAnalysis.ipynb example.
[x] Included detailed comments and documentation in the notebook.
[x] Verified the code's functionality and output.

Feedback and Suggestions
Maintainers are welcome to provide feedback or suggest improvements for this contribution.

Thanks for your PR.
This example does not demonstrate something new compared to other examples in the repo. Hence I will have to close the PR.
If you want to showcase examples using aisuite, you could create a repo of your own and put some interesting examples there. In near term of future, we would have a section in our README that links to examples written by others in their repo. You can submit yours then. Hope this makes sense to you. Thank you again for your contributions and support to the aisuite community.


When the report is empty, the value of payload is the response header.

Describe the bug
When the report is empty, the value of payload is the response header.
I don't understand the intention of this practice.
To Reproduce
Steps to reproduce the behavior:

Get an empty Report
View this ApiResponse.payload

Expected behavior
If the report is empty, I think payload should also be empty, not the response header.

Ok I will have a look and check later.

Hi could you share your python code to "Get an empty Report"?
I don't know how to reproduce your problem.
If I tried a empty report like this:

import logging
import json
from ad_api.base import AdvertisingApiException, Marketplaces
from ad_api.api.sp.reports import Reports

logging.basicConfig(
    level=logging.DEBUG,
    format="%(asctime)s:%(levelname)s:%(message)s"
)

try:

    # report_id = 'amzn1.clicksAPI.v1.m44551.6189D022.4543b157-d833-4bc5-a982-be6ec337b9530'
    report_id = 'empty'

    result = Reports().get_report(
        reportId=report_id
    )
    logging.info(result)

except AdvertisingApiException as ex:
    print(ex)

I get this payload:
{'code': 'NOT_FOUND',              'details': 'Report is not found by provided report identifier: '                         "'empty'.",              'requestId': '8133FSPAEPM4V6MK4XCV'}}

hi, guys! I'm sorry for the mistake in my expression.
I mean the result of the download is an empty list.

https://github.com/denisneuf/python-amazon-ad-api/blob/2cc9be0f789c21e58b0b79d9c2e0bd9d55c7e111/ad_api/base/api_response.py#L5
i think:
self.payload = payload

Got it, already updated. Kindest regards.


Improve rate and delay for polyphase filters

Fixes #233

Codecov Report
Attention: 33 lines in your changes are missing coverage. Please review.

Comparison is base (0a06f68) 75.30% compared to head (9bda46a) 75.47%.

Files
Patch %
Lines

src/sdr/_filter/_polyphase.py
58.57%
29 Missing :warning:

src/sdr/_filter/_design_multirate.py
60.00%
4 Missing :warning:

Additional details and impacted files
@@                Coverage Diff                @@
##           release/0.0.x     #235      +/-   ##
=================================================
+ Coverage          75.30%   75.47%   +0.16%
=================================================
  Files                 48       48
  Lines               2588     2565      -23
=================================================
- Hits                1949     1936      -13
+ Misses               639      629      -10

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


Inconsistency with comments

Hi,
Studying the transform_targets_for_output function of the dataset.py, I realized that the comments don't make much sense.
In the comments are as follows:
# y_true_out: (N, grid, grid, anchors, [x, y, w, h, obj, class])
# grid[y][x][anchor] = (tx, ty, bw, bh, obj, class)

But I have the impression that the first 4 numbers are 2 coordinates, for example:
(x1, y1, x2, y2, obj, class)
To be faithful to the paper, shouldn't it be (tx, ty, bw, bh, obj, class)?
Thanks

I'm pretty sure they are transformed later in model.py loss function, you can double-check on that.


change paths to use 3ds/data/Simple Badge Injector

the /3ds/data/appname/ folder structure has become a bit of a standard for many homebrew like this, so it'd be wise to switch to it since as is, it dumps folders on the root of the SD, which is kind of messy.
If not that's alright too, I'll continue to use my own fork, but here's the change proposal anyway.

https://github.com/cheatfreak47/Simple_badge_injector/releases see here for a build of these changes, works as expected

I changed this so it now dumps into the 3dsx directory instead because things have moved away from using 3ds/data it seems.


Add Lammers2021-libraries

Pull Request
This PR is for a

[x] New Publication(s)
[ ] Correction

For the following list(s):

[ ] ancientmetagenome-anthropogenic (README)
[x] ancientmetagenome-environmental (README, ask @thedir-team-dirt for advice)
[ ] ancientmetagenome-hostassociated (README, ask @thedir-team-bugparty for advice)
[ ] ancientsinglegenome-hostassociated (README, ask @thedir-team-pathopeeps for advice)

New Publication
Publication Information
This pull request is to add samples from the following publication(s):
This is to close #470
Checklist

[x] Publication is published (preprints currently not accepted)?
[x] Checked the publication is not already in the database?
[x] Checked samples in this publication are not previously published data (newly re-sequenced metagenomes are OK!)?
[x] Samples are shotgun metagenomes (hostassociated-singlegenome may also contain whole-genome enriched data)?
[x] Checked the list follows conventions as described in the corresponding sample type's README file (e.g. using ERS/SRS accession codes for ENA/SRA)?
[ ] Changelog is updated to include the publication under 'Added'?
[ ] Pull request has passed validation checks (see automated comment from github-bot)?
[ ] Review requested from corresponding team (see list of lists above for each team)?
[ ] (If applicable) Made a separate PR to add new categories to controlled lists (stored under /assets/enums, e.g. material or archive)? If so the corresponding PR is here:

If you do not know how to check errors in failed validation checks, expand here

Correction
This PR is for

[ ] ancientmetagenome-hostassociated
[ ] ancientsinglegenome-hostassociated
[ ] ancientmetagenome-environmental
[ ] ancientmetagenome-anthropogenic

Reference
This pull request is to correct samples from the following publication(s):
This is to close
Description
The issue is:

Checklist

[ ] Checked the corrected entries follow conventions as described in the corresponding sample type's README file (e.g. using ERS/SRS accession codes for ENA/SRA)
[ ] Changelog is updated to include the publication under 'Changed'?
[ ] Pull request has passed validation checks (see automated comment from github-bot)?
[ ] Review requested from corresponding team (see list of lists above for each team)?

Read length

@jfy133 Just to clarify before i change that, by read length you meant the sequencing_cycles column?

@Darcy220606 correcT!


Add support for opening from memory directly (MMDB_open_memory).

I needed to be able to open the database directly from memory as I link the database into my binary to avoid the open call and managing file system dependencies.

Fixed.

I went with the memory_filename approach as flags is too public, they are set directly by the caller while this is more of an internal flag. I'm also confused about the purpose of flags as they do not seem to be used currently. Also I think it's nice if ->filename is not null after an open as that could be more compatible with existing code.

This doesn't seem like a bad idea. I am not sure why this PR didn't get merged originally, perhaps due to lack of tests. However, it is now quite out of date. I am going to close it, but please feel free to resubmit with tests. I apologize that this sat for so long!


Peer Review - Expand Results & Discussion

Expand Results & Discussion
The "Results & Discussion" section is well-written, but there are opportunities to make it even more insightful:
The discussion on alcohol consumption having a negative effect is brief. Could you explore why this might be the case? For instance, is it related to reduced study time or other associated behaviors? Adding this reasoning would provide more depth.
You mentioned exploring other models but didn’t go into detail about why Ridge Regression might struggle with non-linear relationships. Including a specific example of how non-linearity might affect this dataset would make the discussion stronger and help readers understand potential limitations.
------Similar to the one above-----
Enhance Discussion on Model Limitations:
Add a section discussing Ridge Regression's limitations, like its assumption of linearity and potential multicollinearity issues. This would help readers critically evaluate the results.
Check Ridge Regression Assumptions:
Include a brief explanation verifying that assumptions like linearity and no strong multicollinearity are satisfied. This would justify the choice of Ridge Regression.
Explain Model Choice:
Provide a clearer justification for choosing Ridge Regression over other models. For example, explain how it handles multicollinearity or why it aligns well with the dataset.
Clarify Unexpected Results:
Expand on surprising findings, like studying negatively impacting grades or socializing having a positive effect. Adding hypotheses or explanations would provide more depth.
Make Visualizations Easier to Understand:

Issue resolved here: https://github.com/UBC-MDS/dsci-522-group-23/pull/79


graphite-api_1.0.1-exoscale1400326939_amd64.deb broken on debian wheezy

The documentation suggests graphite-api can be installed on debian wheezy by using the package hosted at https://packagecloud.io/exoscale/community/debian/
9f9574e5 and 37d907b3 are needed for the package to install on this distro, but once this is sorted out, the initscript silently fails to start the service:
# /etc/init.d/graphite-api start
# echo  $?
0

And the reason is:
# /usr/share/python/graphite/bin/gunicorn
/usr/share/python/graphite/bin/python: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.15' not found (required by /usr/share/python/graphite/bin/python)
/usr/share/python/graphite/bin/python: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.14' not found (required by /usr/share/python/graphite/bin/python)

wheezy comes with:
ii  libc6:amd64                  2.13-38+deb7u6      amd64               Embedded GNU C Library: Shared libraries

It looks like the package for wheezy was built on debian testing or one of the more recent ubuntus.
Not a huge issue, as it's trivial to rebuild packages yourself, but it leaves debian wheezy users unable to get started as easily as http://graphite-api.readthedocs.org/en/latest/installation.html suggests.
Once you release 1.0.2, can you please make sure to rebuild the packages on the target platforms ? Thanks :-)

Thanks for the report… The package was built on Ubuntu 14.04. I'll try to publish specific packages for debian and ubuntu next time I make a release :)

Hey Bruno,
I have RedHat src rpms if interested, I use them to deploy in my
environment.
-Chris
On Mar 24, 2015 5:14 AM, "Bruno Renié"<EMAIL_ADDRESS>wrote:

Thanks for the report… The package was built on Ubuntu 14.04. I'll try to
publish specific packages for debian and ubuntu next time I make a release
:)
—
Reply to this email directly or view it on GitHub
https://github.com/brutasse/graphite-api/issues/103#issuecomment-85413406
.

There are now separate builds for debian and ubuntu, closing. https://github.com/brutasse/graphite-api/releases


Add ES translations for Introduction to Azure Static Web Apps

Path to files:
./short/intro-static-web-apps/translations
ES translation:
../translations/es/README.es.md

@alfredodeza

Pull Request for the Issue #111 , review, and let me your comments @alfredodeza .

Now you can check it again please @abrilurena 😃.

I've resolved the new comments, please review @abrilurena

And if exists another workshop to translate, I'd like to help😃.


travis failed on master / not running

CI doesn't appear to be running for my last two PR's (#83, #84 )
travis build looks failed on master
build badge in README is not pointing to master

https://travis-ci.org/HyperionGray/trio-websocket/jobs/453122094
$ make test
pytest --cov=trio_websocket
Traceback (most recent call last):
  File "/home/travis/virtualenv/python3.6.3/lib/python3.6/site-packages/pluggy/__init__.py", line 397, in load_setuptools_entrypoints
    plugin = ep.load()
  File "/home/travis/virtualenv/python3.6.3/lib/python3.6/site-packages/pkg_resources/__init__.py", line 2404, in load
    self.require(*args, **kwargs)
  File "/home/travis/virtualenv/python3.6.3/lib/python3.6/site-packages/pkg_resources/__init__.py", line 2427, in require
    items = working_set.resolve(reqs, env, installer, extras=self.extras)
  File "/home/travis/virtualenv/python3.6.3/lib/python3.6/site-packages/pkg_resources/__init__.py", line 875, in resolve
    raise VersionConflict(dist, req).with_context(dependent_req)
pkg_resources.VersionConflict: (pytest 3.3.0 (/home/travis/virtualenv/python3.6.3/lib/python3.6/site-packages), Requirement.parse('pytest>=3.6'))

I have no idea why travis isn't running on those two PR's.

I think it didn't build because I changed Travis settings. I had both the "branch" and "PR" triggers turned on, but it was building twice for each PR, so I turned off the PR trigger. But I guess the branch trigger doesn't apply to external repos...

I've turned the PR trigger back on just now.

thank you


Feature Request - Stacked script icon for multiple MonoBehaviours on one instance

Feature Description
An option to display a single stacked icon for multiple MonoBehaviours on an instance rather than multiple script icons.
Reason for feature
When all icons are shown, an instance with numerous MonoBehaviours will use an excessive amount of the space an instance has. An option to display a single stacked icon for multiple scripts would be much cleaner. This would also be kept optional.

Implemented and pushed to develop for v0.9.0


Add landcover

Added landcover to the carbon tracker workflow.

Your first PR! Awesome :partying_face:
It's still a draft PR, if it's ready for review (which I assume it is because you asked me to review), you can click the "Ready for review" button to make it a normal pull request.


.async effect

Runs async function with status as result (access to async execution)
let [result, error, loading] = request(e => {
// do request
})

See #36


C# string formatting with negative numbers are not working as expected in net5.0

C# string formatting with negative numbers are not working as expected in net5.0

I have created .net 4.8 framework console app and net 5.0 console app.
Then I tried to set value.ToString(format) in both versions and they are giving different results
Example:
var d = -0.0305441935;
Console.WriteLine(d.ToString("f1"));
In .Net Framework 4.8 console app it gives me result => 0.0
whereas as in net 5.0 console app it gives me result => -0.0

Behavior in both should be same. Look like its broken in net 5.0 version.

As I understand it, this was an intentional breaking change to make sure strings more accurately represent floating point numbers, and also to follow the IEEE 754 standard. For more details, see https://github.com/dotnet/runtime/issues/28941. (That issue does not discuss format specifiers like "f1", but I don't think that makes a difference here.)


Add a --watch parameter

hey -- thanks for such a great project.
we have a script that may need to restart to pick up files it hasn't imported yet, so i added a --watch CLI switch. i did a smoke test and it worked, but let me know if there's a better way to do this.

What files do you want to watch with this?

this is for our unit test runner. when our test runner starts up, it looks at the untracked and modified files in the repo to determine which tests to run by walking the dependency graph. since the set of tests that we want to run may change, ts-node-dev might not be watching the correct set of flles as it wasn't required in the previous test run. i could put this together with nodemon and ts-node, but i want to take advantage of ts-node-dev's caching for fast reloads.

 Just want to understand your use case better.

sure! i'm watching our src/ directory to pick up all source file edits, similar to how you'd use nodemon. the advantage of using ts-node-dev vs ts-node + nodemon is that the startup time is much faster due to the compilation caching.

Will need to add recursive param to watch for watching folders, so will test how it works and release.

See also https://github.com/fgnass/node-dev/issues/207 for use cases for --watch.
This PR may be more appropriate for node-dev itself rather than ts-node-dev.

@buyle ts-node-dev based on a fork anyway and has a number of fixes for original code,
@petehunt
anyway I actually published a version with added param, you may check.


Typo: schema vs database in README (esp. DataBricks)

    salesforce_schema: your_database_name     salesforce_database: your_schema_name
Is that correct? I would have expected:
    salesforce_schema: your_schema_name     salesforce_database: your_database_name
Also, do I always need to specify this explicitly?
The package install default appears to have database as 'Undefined', which is causing problems with my Databricks warehouse.  Is that even supported?

Invalid sources config given in models/src_salesforce.yml @ sources: {'name': 'salesforce', 'schema': 'salesforce', 'database': Undefined, 'loader': 'fivetran', 'loaded_at_field': '_fivetran_synced', 'freshness': {'warn_after': {'count': 24, 'period': 'hour'},

FYI, testing with v0.5 and dbt-databricks works once I add that workaround:
vars:     salesforce_schema: salesforce     salesforce_database: ''
It would be great to add that snippet, as it may be the only fix most people need
we use the following snippet to ignore a database config "{% if target.type != 'spark'%}{{ var('salesforce_database', target.database) }}{% endif %}"


Failed to run on emulator after upgrading from 0.42.3 to 0.43.0

After upgrading from 0.42.3 to 0.43.0 I got this error when using ListView:
undefined is not a function (evaluating 'this.updater.enqueueCallback(this, callback, 'setState')')
Does anyone got the same error like this?
Additional Information

React Native version: 0.43.0
Platform: Android
Development Operating System: Windows
Dev tools: Android Studio version

Hey, thanks for reporting this issue!
It looks like your description is missing some necessary information. Can you please add all the details specified in the template? This is necessary for people to be able to understand and reproduce the issue being reported.

I got this same issue when use ListView

Did you find a solution? I'm getting this error also after a fresh install of react native. It works fine on my other computer and nothing in the project has changed.

@rom1k Could you say what version the old version of node was? I have v8.3.0 and still get the same error.


CSDL to OpenAPI Translation of Responses with Primitive Values

Describe the bug
The generated OpenAPI document does not accurately represent response objects for operations that return single primitive type values, e.g., POST https://graph.microsoft.com/v1.0/users/{id}/revokeSignInSessions. Quoting from the OData specification:

A property or operation response that is of a primitive type is represented as an object with a single name/value pair, whose name is value and whose value is a primitive value, i.e.,

{
  "@odata.context":"https://graph.microsoft.com/v1.0/$metadata#Edm.Boolean",
   "value":true
}

To Reproduce

CSDL

Current OpenAPI Document

Expected behavior
The OpenAPI document should represent response objects for operations that return a single primitive type as an object with a single name/value pair, whose name is value and value is a primitive type.

Additional context
Blocks https://github.com/microsoftgraph/msgraph-sdk-powershell/issues/834.
Also affects:

/deviceManagement/applePushNotificationCertificate/downloadApplePushNotificationCertificateSigningRequest()

Was fixed by https://github.com/microsoftgraph/msgraph-sdk-powershell/releases/tag/1.10.0


complete frontend project member list

member list with audit information.

should fix the close buttons of the delete forms. it's neither possible to close the modal when clicking anywhere on the screen. and they look ugly!
next, fix the MemberServiceIT.


Issue #817 Incorrect Line Break in KO onboarding

Do not merge yet. The breaks in the word need to be verified for someone who speaks KO. It is unclear whether or not the break/space between characters are problematic.


Params to turn off next/previous page - and breadcrumb

I think this is all good!
re: #136

tweaked to allow
disableBreadcrumb: true
disableNextPrev: true

In page params or site params

Sorry, I'm super late. See my comments :)

And btw, could you update the documentation (English part, I will do the french one ^^) with your changes ?

Site params: http://learn.netlify.com/en/basics/configuration/#global-site-parameters
Page params: http://learn.netlify.com/en/cont/pages/#front-matter-configuration

Thanks again :)

Can we merge this in as a sitewide setting and add per-page override to the backlog? I've tried implementing this but can't solve without having a disableNextPrev and a enableNextPrev.


chore(release): v0.16.0 🎉

:robot: I have created a release beep boop
0.16.0 (2024-05-11)
Features

alias: support DomainObject.alias declaration (#37) (7348ae2)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/ehmpathy/domain-objects/releases/tag/v0.16.0 :sunflower:


Add LocalService SID and Generic well known SID function, downgrade to 1.21.8

Adds a dedicated function for getting the LocalSystem SID, which is typically added to files in addition to the administrators SID. Additionally adds a function to get generic SIDs based off of the well known SID types in the windows library.
The go version has been downgraded from 1.22.1 to 1.21.8 in order to satisfy build requirements for k3s. The latest k8s version (1.29) must be built with 1.21.8.
Additionally this PR adds a Mkdir function to create windows directories using security attributes. This is a utility function that ensures that directories are created with the correct ACLs as opposed to patching directory ACLs after they have been created

@HarrisonWAffel I pulled in your changes directly into the main branch, made some refactored changes to get Mkdir to use Apply under the hood as well.


Optimize logic of DubboBeanDefinitionParser.java

What is the purpose of the change
Make code of process callback of method cleaner in DubboBeanDefinitionParser
Brief changelog
DubboBeanDefinitionParser.java
Verifying this change
XXXXX
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Run mvn clean install -DskipTests=false & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[ ] If this contribution is large, please follow the Software Donation Guide.

there is one problem, i will commit again

there is one problem, i will commit again

It's ok

Codecov Report

Merging #4770 into master will decrease coverage by 0.03%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #4770      +/-   ##
============================================
- Coverage     63.96%   63.93%   -0.04%
+ Complexity      452      451       -1
============================================
  Files           769      769
  Lines         33179    33167      -12
  Branches       5229     5227       -2
============================================
- Hits          21222    21204      -18
- Misses         9540     9542       +2
- Partials       2417     2421       +4

Impacted Files
Coverage Δ
Complexity Δ

...onfig/spring/schema/DubboBeanDefinitionParser.java
65.86% <100%> (-1.87%)
0 <0> (ø)

.../remoting/transport/netty4/NettyServerHandler.java
61.9% <0%> (-9.53%)
0% <0%> (ø)

...ng/transport/dispatcher/all/AllChannelHandler.java
51.42% <0%> (-5.72%)
0% <0%> (ø)

.../org/apache/dubbo/remoting/ExecutionException.java
15.78% <0%> (-5.27%)
0% <0%> (ø)

...e/dubbo/remoting/transport/netty/NettyChannel.java
57.64% <0%> (-4.71%)
20% <0%> (-1%)

.../apache/dubbo/remoting/transport/AbstractPeer.java
58.69% <0%> (-4.35%)
0% <0%> (ø)

...he/dubbo/remoting/transport/netty/NettyServer.java
69.64% <0%> (-3.58%)
8% <0%> (-1%)

...c/main/java/org/apache/dubbo/rpc/RpcException.java
80% <0%> (-3.34%)
0% <0%> (ø)

...exchange/support/header/HeaderExchangeHandler.java
64.75% <0%> (-2.46%)
0% <0%> (ø)

.../java/org/apache/dubbo/config/ReferenceConfig.java
60.58% <0%> (-0.37%)
0% <0%> (ø)

... and 5 more

Continue to review full report at Codecov.

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


Skynovels

Source name
Skynovels
Source link
https://www.skynovels.net/
Source language
Spanish
Other details
"18+/nsfw = no
Acknowledgements

[X] I have checked that the extension does not already exist by searching the extension listing and verified it does not appear there.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a meaningful title with the source name.
[X] I will fill out all of the requested information in this form.

Tachiyomi / Mihon / forks don't support novels


Aws Deployment Prep

Naming: AWS-deployment-prep
Reviewers: minimum 2
Description:
What is the motivation for changes?

Cleaned and reorganized code per Data Science Lead Robert Sharp guidance
Since AWS will only deploy the app folder, we only included relevant files inside of the app folder that's needed for
deployment
What specific changes were made?
removed old unneeded files from code base including:
- textmatcher.py
- training_data.py
- requirements.txt (removed since intended for heroku)
- reddit.py
- db.py
- twitter.py
- cleaned misleading comments in:
- main.py
- tweet_reachout.py
- helper_funcs
moved the following files to Scripts file and added a readme to Scripts folder for currenlty unused code for the app
- create_db_tables.py
- tweet_reachout.py
- twitter_bot.py
updated main.py
- so that it removed the old db, reddit, and twitter id lookups as team determined functionality wasn't nessessary
- added a GET endpoint to return frankenbert predictions on user inputted text
updated the get_rank_of_force function in helper_func.py so that it doesn't clean input text since we don't clean the input text
for our training data
minor stylistic coding updates on various files

Submitter Checklist:

[X] Small scope (1-2 components)
[X] Not too many concerns in a single commit
[X] Remove extraneous comments (code in comments), print statements, to-dos, console.logs, extra fluff
[X] Atomic, descriptive commit messages
[X] Consistent formatting

Reviewer Checklist:

[ ] Small scope (1-2 components)
[ ] Comments, print statements, to-dos, console.logs, extra fluff were removed
[ ] Give feedback no matter what conversations happened elsewhere, document here
[ ] Comment on the line of code itself (blue plus box) in files changed tab

Looks great! Thanks, Eric!


Armada 668 Update Jobbergate CLI for changes to application create workflow

What
Update Jobbergate CLI for changes to application create workflow (see #128):

Remove upload file validation from the CLI.
Remove compression of the upload files into tarballs.

Update Jobbergate slurp:

Add jobbergate-api as a requirement, so some components can be reused and code duplication is avoided.
Implement new logic to handle file uploads.

Update Jobbergate API:

Refactor application file management in a class, making it modular and reusable.
Fix a bug when writing application files to S3 and add tests to cover the issue in the future.

Why
The creation workflow for applications has changed dramatically. Update the Jobbergate CLI to fit.
Task: https://app.clickup.com/t/18022949/ARMADA-668

Peer Review
Please follow the upstream omnivector documentation concerning
peer-review guidelines.

Task linked: ARMADA-668 Update Jobbergate CLI for changes to application create workflow

The code here was refactored, so it makes more sense to close this PR and do the review on the following:

#137


JWT should not be stored in localstorage

It's insecure. See https://blog.logrocket.com/jwt-authentication-best-practices/
More generally, it's worth being acutely aware of how difficult security is in JS. Much of the following article does not apply to us, and some of it is out of date, but some of it is relevant and it's a fun read. :)
https://www.nccgroup.trust/us/about-us/newsroom-and-events/blog/2011/august/javascript-cryptography-considered-harmful/

Next step is to re-factor the code. This is a known issue 👍

Yeah if I drop my JWT into jwt.io I can see it's data....looks like it carries my ETH address and an expiry.

will be addressed with latest release, hopefully end of this week.


Fix issues related to having catalog_name in identifier

An attempt to fix #742 without removing the catalog_name. This may serve as a temporary solution if we finally decide to
do https://github.com/apache/iceberg-python/pull/963 in a later release (or not remove). Otherwise #963 should be preferred

+1 to this solution for 0.7.0 @HonahX . This is a creative way to mitigate the issue for the release in the short term. Thank you for taking your time to put together this solution so quickly!


fix the not changing image bug

Description:
i fix the changing of image according to check the button .

This is PR of #191

@yatulearn please check it . and label it

the issue has already been fixed by our team, so I am closing the pr.


Support for vscode's "hide" option

I added a field to the TemplateDefinition called hide, that is used in the run_template function to filter templates and only show to the user the not hidden one.
I tried reading the code base and it seemed the most concise yet aligned solution I could come up with.
Please let me know if there are any chore addition needed, it didn't seem the documentation should be updated since it is an "internal" change nothing new should be exposed to the user.
I hope you'd like it and maybe it can close issue #269

Thanks for the PR!


Add function to generate UUID

Tested with Cygwin, MinGW, MSVC and GCC on Linux.
TODO: Add libRpcrt4.a to the Linux libraries

The only concern I have that you previously could compile ModelicaUtilities.h and run MDD_utilitiesLoadRealParameter on a Mac, but you will now get an unsupported platform compiler error. Maybe it is better to have separate header file for the new function.

I get strange errors for OMC on Windows. I also discovered that ModelicaFormatWarning and ModelicaWarning is not (yet) supported by OMC, but I will make that note to the corresponding PR

cmake rule to build dummy libraries missing.

Resolved by 23d2c3fb812b93651a73d7428cbce0cf0880ae71.

In Include/MDDUtilities.h: Is the pragma #pragma comment( lib, "Rpcrt4.lib" ) really needed.

Yes, still required, since it is a workaround for C-Code Export of SimulationX, where Rpcrt4.lib cannot be found in the (default) Library directory specified by Modelica_DeviceDrivers.Utilities.Functions.generateUUID. Unfortunately SimulationX removes these library dependencies to non-default MSVC libs during translation, but having it in the header file as MSVC specific pragma fixes the linking again. See also section 3.1.1 in the article on modelica2017 branch.

OMC 1.12 Windows 64-bit
undefined reference to `pthread_once'

Hm, #114 is still not resolved.

I also discovered that ModelicaFormatWarning and ModelicaWarning is not (yet) supported by OMC, but I will make that note to the corresponding PR

Resolved by #246.


Add span with command-line-(input|output) class to command-line plugin lines

Currently it is not possible to distinguish between input and output, unless all input has some kind of highlighting on it.  This change wraps each line with a span and class to indicate whether it is input or output.
Reasons for needing to do this so that it is possible to:

Style the output to look different to the input, e.g. lighter colour
Set user-select: none; for output lines
Maybe add a toggle to show/hide output lines.

For example

			echo
			"hello" plain
			"world"

		hello plain world

			echo
			"

					$(
					date
					)
				"

		Fri 28 Jan 21:06:47 GMT 2022

The above HTML was from

Accepted your css class name change.  I have re-generated the minified js.

Thank you for contributing @at055612!

Thanks for merging this.


Spanish translation

Are you interested in a spanish translation? I can send a PR.

Absolutely! Please do :+1:


consistent naming of Environment Variables

The Environment variables dont' have a consistent naming.
In the table below are the names in the config files (in modelConfiguration) and in the code. The FACSIMILE format generally follows the naming of the config files:

config
code

M
m

TEMP
temp

H2O
h2o or H2o

DEC
dec

PRESS
pressure

RH
rh

BLHEIGHT
blh

DILUTE
dilute

JFAC
jfac or JFac

ROOFOPEN
roofOpen

This could be only a cosmetic thing (as Fortran is case insensitive, in which case is only relatively important) or it could be useful to have different names for the input options in the config files and for the variables in the code. Then what is the relationship the names in the FACSIMILE file?
I don't have a preference, but it should be consistent. What is best practice?
(this may be related to issue #3)

This does not affect the way thye code works, so it is to be addressed in the wiki: https://github.com/AtChem/AtChem/wiki/Style-Guide


Add spec

Coverage increased (+17.04%) to 89.08% when pulling 28f104e2ff2411f72ba2377adbd832c74e55d347 on scaffold-20150413 into a546aa1be2a703b5a1afee680ee500ee06037ad3 on master.