Unable to download tokenizer. S3 link seems to be inaccessible

I tried installing the npm package. One of its step includes download of the tokenizer from https://tokenizers-releases.s3.amazonaws.com/node/0.7.0/index-v0.7.0-node-v83-linux-x64-glibc.tar.gz
This seems to be inaccessible.
I am using node v12.
Can you please help me with this?

Bumping this issue, getting the same issue:
 ERR! command failed
npm ERR! command C:\WINDOWS\system32\cmd.exe /d /s /c node-pre-gyp install
npm ERR! 403 status code downloading tarball https://tokenizers-releases.s3.amazonaws.com/node/0.7.0/index-v0.7.0-node-v102-win32-x64-unknown.tar.gz
npm ERR! node-pre-gyp info it worked if it ends with ok
npm ERR! node-pre-gyp info using<EMAIL_ADDRESS>npm ERR! node-pre-gyp info using<EMAIL_ADDRESS>| win32 | x64
npm ERR! node-pre-gyp WARN Using needle for node-pre-gyp https download
npm ERR! node-pre-gyp info check checked for "C:\Users\admin\git\huggingface-qa\node_modules\tokenizers\bin-package\index.node" (not found)
npm ERR! node-pre-gyp http GET https://tokenizers-releases.s3.amazonaws.com/node/0.7.0/index-v0.7.0-node-v102-win32-x64-unknown.tar.gz
npm ERR! node-pre-gyp http 403 https://tokenizers-releases.s3.amazonaws.com/node/0.7.0/index-v0.7.0-node-v102-win32-x64-unknown.tar.gz
npm ERR! node-pre-gyp ERR! install error

Any updates with this issue ? I am facing the same issue. Kindly update the package with the new link.

@SaulLu @Narsil Do you know if Node is still supported as a compilation target from huggingface/tokenizers? Also we might want to transfer this issue to that repo.

Unfortunately I don't know either 😞

tokenizers does support and provide build targets for node, 12 and 14 (not 16 atm iirc linked to us needing to do a big upgrade on the bindings side).
Everything is available through npm install tokenizers normally. Not sure what that old s3 link is.

Downgrading to node 14 does seem to work but, as @Narsil mentioned, newer versions don't work. I also get the following deprecation warning:
npm WARN deprecated<EMAIL_ADDRESS>This version has been yanked and won't be supported
Can someone provide an update on this package's future?

@creatorrr ,
PRs are welcome. but updating to new node versions require updating to neon==latest which requires a lot of changes(didn't try yet, but it felt like a rewrite basically).
I am not going to lie, no one is dedicated specifically to tokenizers in general (as it works mostly as intended for transformers which is our biggest user). So spending time rewriting for node is not super high on the prio list right now.
PRs are very welcome if you feel like it !

hey is this repo still active? Would love to help get this working again. Trying to get an application working with this and jumping through hoops with Node/Python just to get stuff installed.

Last commit is from 3 years ago...

@Narsil i noticed, but I see activity in issues as recent as last year. Is this package considered completely abandoned at this point?

Definitely not actively maintained. tokenizers and its node bindings are, but they are in maintenance mode only (until someone wants to help redo the bindings with latest neon)

we can archive this repo btw maybe

@Pierrci Are you ok with archiving this ?

if this repo is being archived, is there a newer, more reliable open source tool I can use to achieve the same behavior?

For those looking for an alternative, I found @tensorflow-models/qna. Does the same thing!


Using lang to specify locale not working?

What version of Citation.js are you using?
0.6.2
How are you running Citation.js (npm package, browser bundle, self-rolled bundle, webpack)?
npm package with webpack
Where are you running Citation.js? * The OS usually will not matter much, but the browser really does.
PubPub / Chrome latest
What is your input data and code?
Input Data:
{
  type: 'article-journal',
  title: 'Hmm ok\n',
  author: [Array],
  editor: [Array],
  'container-title': 'Journal of KFG',
  containerDoi: '10.21428/0417b0c0.2ef9383d',
  ISBN: undefined,
  ISSN: null,
  edition: undefined,
  volume: '',
  issue: '',
  publisher: '',
  DOI: '10.21428/0417b0c0.2ef9383d',
  issued: [Array],
  note: 'http://localhost:9876/pub/d9511nz9',
  URL: 'http://localhost:9876/pub/d9511nz9',
}

Cite.format('bibliography', { format: 'html', template: 'harvard', lang: 'en-US' });
What did you expect to happen, and what actually happened?
Expected:
Dent, A. et al. (2022) 'Hmm ok', Journal of KFG [Preprint]. Edited by I. Reynolds. doi:10.21428/0417b0c0.2ef9383d.
Got:
Dent, A. et al. (2022) "Hmm ok," Journal of KFG [Preprint]. Edited by I. Reynolds. doi:10.21428/0417b0c0.2ef9383d.
I also did the same passing fr into an APA style with an editor, and did not see any of the French editor terms éd. with the accent vs. ed.
When I manually downloaded the harvard-cite-them-right csl and changed the default locale to en-US, the citation appeared as expected.
This appears to be a known-ish thing (https://github.com/citation-style-language/styles/issues/5956), but Zotero, which also uses citproc-js, doesn't have the issue, and manually changing the locale in the CSL file works, so my guess is this comes down to citation-js's locale handling.

Oh I see, it is a bug in Citation.js. It passes a language by default but that overrides the default-locale.

Fix released in 0.6.3

Confirm it works well, but there's one potential wrinkle that I'm not sure is intended that I wanted to give you a heads up on. Now, when you don't specify a language for Harvard, it throws the error citeproc-js error: unable to parse XML input. May be due to Harvard having a default locale set? Does not happen with APA.
Input
{
  type: 'article-journal',
  title: 'Hmm ok\n',
  author: [Array],
  editor: [Array],
  'container-title': 'Journal of KFG',
  containerDoi: '10.21428/0417b0c0.2ef9383d',
  ISBN: undefined,
  ISSN: null,
  edition: undefined,
  volume: '',
  issue: '',
  publisher: '',
  DOI: '10.21428/0417b0c0.2ef9383d',
  issued: [Array],
  note: 'http://localhost:9876/pub/d9511nz9',
  URL: 'http://localhost:9876/pub/d9511nz9',
}

Cite.format('bibliography', { format: 'html', template: 'harvard' });

Yeah normally unavailable locales are handled but in this case Citation.js does not know about the locale so it cannot handle it directly. For compatibility I made it fail silently in 974ea05bd65f7f0753e83d0544d47a818ed3e935. This makes it harder to debug of course but that is the trade-off until the next major version.


script: Added support for Ubuntu 18.04 in the install script.

Closes https://github.com/flynn/flynn/issues/4392

@philiplb Thanks! Have you tested this on 18.04?

Hi, yes, I launched an EC2 instance with Ubuntu 18.04 and ran a slightly modified version of this script with the FLYNN_HOST_CHECKSUM from the install script from dl.flynn.io It went through successfully. I didn't launch the flynn-host afterwards though.

Thx @AnikHasibul. :-)

Closed due to https://github.com/flynn/flynn/commit/8ac5d71325e4c1efcb6008bf6377995370f7837f


Add Step component to getHeadings function

Signed-off-by: Jay Clark<EMAIL_ADDRESS>Fixes #59
Take a look and let me know what you think! The three main judgment calls I made that may or may not be correct:

Making a more involved regex to accommodate 'number' and 'title' of the component possibly being in a different order
Ignoring a Step component if it is missing either the number or the title prop.
Ignoring single quotes. Every Step component so far has used double quotation marks for strings. However, I do see single quote marks used occasionally throughout the repo. if the style guide/eslint config file changes in the future to enforce single quotes (or if an author uses single quotes for a Step component), that would cause the regex to no longer work.

Also, added a small tweak to WrittenBy.js to prevent the site from crashing if no social links exist (which was happening for me on the page that was used to illustrate the issue.)

Awesome work! Here's my responses:
Making a more involved regex to accommodate 'number' and 'title' of the component possibly being in a different order
This is definitely the ideal solution here.
Ignoring a Step component if it is missing either the number or the title prop.
Since each Step should have both and we are using TypeScript, I think an easier (and better) solution would be to convert the Step component from js to ts and make the props required.That way, the writer will be warned in their IDE that both props are required and it won't compile.
Ignoring single quotes. Every Step component so far has used double quotation marks for strings. However, I do see single quote marks used occasionally throughout the repo. if the style guide/eslint config file changes in the future to enforce single quotes (or if an author uses single quotes for a Step component), that would cause the regex to no longer work.
If possible could you make it work with single quotes too? Right now there is no style guide and you mentioned that there is both currently. I honestly don't have a preference of single or double quotes and trying to enforce one would be pointless and very difficult so I think it would be best to accommodate both.
My last request would be to remove the numbers from the Accordion dropdown. We just need the title showing!

Awesome work! Here's my responses:
Making a more involved regex to accommodate 'number' and 'title' of the component possibly being in a different order
This is definitely the ideal solution here.
Ignoring a Step component if it is missing either the number or the title prop.
Since each Step should have both and we are using TypeScript, I think an easier (and better) solution would be to convert the Step component from js to ts and make the props required.That way, the writer will be warned in their IDE that both props are required and it won't compile.
Ignoring single quotes. Every Step component so far has used double quotation marks for strings. However, I do see single quote marks used occasionally throughout the repo. if the style guide/eslint config file changes in the future to enforce single quotes (or if an author uses single quotes for a Step component), that would cause the regex to no longer work.
If possible could you make it work with single quotes too? Right now there is no style guide and you mentioned that there is both currently. I honestly don't have a preference of single or double quotes and trying to enforce one would be pointless and very difficult so I think it would be best to accommodate both.
My last request would be to remove the numbers from the Accordion dropdown. We just need the title showing!

Great, thanks for the feedback! I've made those changes, let me know what you think.

Great, thanks for the feedback! I've made those changes, let me know what you think.

Everything looks good! I would also like to add a contributors section to the site to highlight those who have contributed (code or article). This will replace the authors page. I will make those changes in another branch but for now could you add a mdx file with your info to the authors folder?

Everything looks good! I would also like to add a contributors section to the site to highlight those who have contributed (code or article). This will replace the authors page. I will make those changes in another branch but for now could you add a mdx file with your info to the authors folder?


Support now alias shortcut

[ ] PR ready?

What's this PR do?
Add support for reading the default alias target from a package.json's config.alias property.
Running now alias will look for the latest deployment (and throw if there isn't one) and then look for the config.alias to allow the user to shortcut the current now alias <deployment> <target>.
Related ticket: https://github.com/zeit/now/issues/68
Note: I'll likely rebase this PR into a single commit for this feature…unless I hear otherwise.

Superseded by #93


Feat:Adding test files

Describe the feature
Wish to add test files for other scrapping functions as well currently there exists only 18-20 test files while there are so many scrapping modules available whose test files do not exist so want to add them as a contributor.
Add ScreenShots
Just like these--

want to add test files for other scrapping modules as well.
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'24 contributor
[X] I want to work on this issue

I can not assign someone to add all the test cases, you can pick one module and add test file of it.


Add support for skipping integration test based on change directory

Description

The current configuration setup for CFT CI based repos doesn't check the files that were changed during the last commit; thus the pipeline is kicked off on every commit
Ideally we would want to be able to specify that a certain CI pipeline/step should run only when changes are made to a specific subdirectory or file

Example:

anthos-samples repository uses the CI pipeline provided by CFT.
With every change the integration test steps in the cloudbuild yaml are run
The integration tests are a time consuming affair and we would like to control when they are triggered based on what files are changed
Ideally as repo owners we should be able to configure when to run a certain build steps in terms of folder changes

At first I was thinking you could just do this in Cloud Build, but it would require many different triggers for parts of the repo so I do think adding this to the CLI could improve UX.
How about we add it to the BlueprintTest yaml. For example:
apiVersion: blueprints.cloud.google.com/v1alpha1
kind: BlueprintTest
metadata:
  name: quuz
spec:
  when:
  - changed:
     - "directory/regex/(.+)"

@Shabirmean Would that work for you?

@morgante - I am not fully aware of how this becomes part of the CI pipeline. But this looks like something that can work. Ideally, this would be something that repo owners can place in their repos and have control of, instead of having to open a change with the Blueprints team.

Yes, that is a yaml file you place in your repo.

@morgante WDYT about leveraging included_files as a solution until we add this to the CLI. We can store a map of repo_name => included_file list and set that during trigger creation if a value exists in map. This would still be managed in this repo.

@bharathkkb Wouldn't that still require creating many triggers for a particular repo? It could work as a workaround, but you'd probably want to make it a map of lists repo_name => [included_files_1, included_files_2] where a separate trigger is created for each included files list.

@morgante I believe the provider accepts a list input for included_files so we can pass in the list directly to a single trigger resource.
https://github.com/hashicorp/terraform-provider-google/blob/b5b0438689c8a10fad804bdbc59ff45afa8aeb33/google/resource_cloudbuild_trigger.go#L838

@morgante I believe the provider accepts a list input for included_files so we can pass in the list directly to a single trigger resource. https://github.com/hashicorp/terraform-provider-google/blob/b5b0438689c8a10fad804bdbc59ff45afa8aeb33/google/resource_cloudbuild_trigger.go#L838

That should work.


Native Editor - Cell command state input/output styling

[ ] Indent selection bar but not output
□ 1 day
[ ] Change background color for output
□ 1 hr
[ ] Draw / colors states
□ 4 hr
[ ] Execution count colors
§ 1 hr

Validated


Moon rendering on top of foreground geometry

The black dot on the surface of California in the below picture is the moon being rendered through Geometry; for some reason it is being drawn on top (though it correctly goes behind the ellipsoid).

Sandcastle link to the demo.

This seems to be fixed (though I'm not sure what changed).  @emackey can you still reproduce this?

Never mind.  This still happens but not in Chrome.  It does happen in IE.  Maybe a missing extension in IE?

I think this is because moon is updated in updatePrimitives as a generic primitive, instead of in executeCommands like the special case stars, atmosphere, and sun so it is rendered after the atmosphere.  The downside of moving it is that we would lose its depth, but I don't think that is a problem for our current use cases.
@bagnell what do you think?


Add a markdown renderer

cc @jasongrout

👍 I've used marked for a while. I think it's pretty good.


How to return all results or certain number of results?

Hi
Is there to control the number of results and list them?

I have the same question

it can be done using the parameter "&start=4 " it will give you results from 4 to 7 and use " &start=8" to get results from 8 to 11 and so on
I made some simple code to automate such process
`import requests
import json
class GoogleSearch:
def init(self, query, pages=1):
self.query = query
self.pages = pages
self.search_link = 'http://ajax.googleapis.com/ajax/services/search/web?v=1.0&q=%s' % self.query
self.results = []
self.links = []
def search(self):
    results = []
    links = []

    for i in range(self.pages):
        search_link = self.search_link + '&start=%i' % (i * 4)
        search_resp = requests.get(search_link)
        response_dict = json.loads(search_resp.text)
        results.append(response_dict['responseData']['results'])

    for r in results:
        for x in r:
            links.append(x['url'])
    self.results = results
    self.links = links`

search_query = GoogleSearch();
search_query.init("test", 2000);
search_query.search();
I got this:
Traceback (most recent call last):   File "observe.py", line 32, in <module>     search_query.search();   File "observe.py", line 21, in search     results.append(response_dict['responseData']['results']) TypeError: 'NoneType' object has no attribute '__getitem__'
Can you show us how to print the links?

the function init in any class is preserved function to initialize the class so when you create an object of the class which you named " search_query " you have to pass values for parameters included in " init " function
this is a sample code usage for the class
search_query = GoogleSearch(query="your search query here", pages=2) # add the number that fits
search_query.search()
print(search_query.links)

TypeError: this constructor takes no arguments

did you copy my code?
give me your full code here

yes, I just used your codes.
import requests
import json
class GoogleSearch:
def init(self, query, pages=1):
self.query = query
self.pages = pages
self.search_link = 'http://ajax.googleapis.com/ajax/services/search/web?v=1.0&q=%s' % self.query
self.results = []
self.links = []
def search(self):
results = []
links = []
for i in range(self.pages):
    search_link = self.search_link + '&start=%i' % (i * 4)
    search_resp = requests.get(search_link)
    response_dict = json.loads(search_resp.text)
    results.append(response_dict['responseData']['results'])

for r in results:
    for x in r:
        links.append(x['url'])
self.results = results
self.links = links`

search_query = GoogleSearch(query="test", pages=2000) # add the number that fits
search_query.search()
print(search_query.links)

you have to copy paste the code to avoid typing mistakes, I can see function init without the " __ " two underscores before and after which is mandatory

File "observe.py", line 32, in
search_query.search()
File "observe.py", line 21, in search
results.append(response_dict['responseData']['results'])
TypeError: 'NoneType' object has no attribute 'getitem'

Unfortunately Google has stopped this API, I got this response
This API is no longer available.
alternatively you can use web scrapping tools just like the below example
import requests
from bs4 import BeautifulSoup

resp = requests.get('https://news.google.com/news/section?cf=all&ned=us&q=nfl').text

soup = BeautifulSoup(resp, 'html.parser')
# print(soup.find_all('a'))
links = []
for link in soup.find_all('a'):
    url = str(link.get('href'))
    if url.startswith('http') and ('google.com' not in url) and 'youtube.com' not in url and \
            'googlenewsblog.blogspot.com/' not in url:
        links.append(url)

links = set(links)
print(len(links))

you can replace "&q=" parameter's value with your search query, in this example I am searching for " nfl " you can change as you need

This departs from this googlesearch package. Too many fragmented solutions. Can we clean it up?

I didn't understand what you mean

The API this module used is no longer available, googlesearch has been now pulled from PyPI. It would need a complete overhaul and rewrite to use the new google API. The last beautifulsoup entry can be helpful if someone is searching for a loosely related alternative, but I am closing this issue, because it is relevant to a now defunct version (due to the API change).


Search inside merged tasks

The current UI searches only what is visible. That means if we have merged tasks, there is no way to search inside them. One request we get regularly is to search inside of merged tasks, even at zoom levels where those tasks are not visible. This is potentially very expensive so it will need to be behind an explicit option.

Ooph, yeah, I'm definitely dealing with this currently. I was wondering why some of my searches were not showing certain kinds of meta-tasks that I knew to be there, but then when I zoomed in they suddenly appeared in the search.


feat: adds SuperOwnable

Description
SuperOwnable contract to manage cross chain ownership.

Tests

Additional context

Metadata

#252

tagging @tynes here


store.subscribe not being triggered

Hi,
I'm using vuex-persistedstate and I need to subscribe to the "value resetting event".
I added store.subscribe to my store but it is not being triggered when the values are put back after refresh.
Amu I missing something?
Thanks

@LeonLiber can you please provide a https://jsfiddle.net/ with your situation / example of your setup?

Hi @robinvdvleuten, It's a bit of a complicated setup for jsfiddle so I'll try to post it here.
The core of the store looks like that:
import Vue from 'vue';
import Vuex from 'vuex';
import persistedState from 'vuex-persistedstate';

Vue.use(Vuex);

export default new Vuex.Store({
  plugins: [persistedState({
    paths: ['user'],
  })],
  state: {
    user: { },
  },
  getters: {
    isUserLoggedin(state) {
      return Object.keys(state.user).length > 0;
    },
    user(state) {
      return state.user;
    },
  },
  mutations: {
    SET_USER(state, userData) {
      state.user = userData;
    },
    UNSET_USER(state) {
      state.user = {};
    },
  },
  actions: {
    setUser({ commit }, userData) {
      commit('SET_USER', userData);
    },
    unsetUser({ commit }) {
      commit('UNSET_USER');
    },
  },
});

I want to subscribe to any change "resetting" of the user so I can set some default http headers with the proper user token.
This is what I tried:
 store.subscribe((mutation) => {
   console.log('MUTATED', mutation);
});

And I get no log when user is re-set after page refresh.
Thanks you

@LeonLiber so as I try to understand your issue, you are trying to set custom axios headers when your app is initialised right?

Correct. The token I'm going to store in the header is coming from the persistent store so I need to act on the state "rehydration"

@LeonLiber why not fetch the correct header from the store when initializing your app (so after the store is dehydrated). Please see both urls for an example;
https://github.com/robinvdvleuten/vue-auth0-vuex/blob/master/template/src/router/index.js#L36
https://github.com/robinvdvleuten/vue-auth0-vuex/blob/master/template/src/router/index.js#L9
This onEnter hook dispatches an event which will fetch a new token. You can use the same pattern for setting the header on axios by fetching the value from the store.


Request: support new Spotify web player

Logged into Spotify web player after using the desktop app for a new days, turns out I have been "selected to test" the new web player UI at https://open.spotify.com:

So, how should I enable this new web player?

So, as I understood, this is just an UI for Spotify Desktop app, right? The Spotify app is able to scrobble tracks by itself. Is there a sense to make a connector for this web UI? Have you any use cases of this?

So, as I understood, this is just an UI for Spotify Desktop app, right?
This is a new web player UI. I don't know if you have been selected for testing this new UI, but it looks like they are doing A/B testing on the new UI currently:
https://community.spotify.com/t5/Other-Partners-Windows-Phone-etc/New-Web-Player-is-garbage/td-p/1559947

Here is a longer thread: https://community.spotify.com/t5/forums/v3_1/forumtopicpage/board-id/001/page/1/thread-id/172874

The Spotify app is able to scrobble tracks by itself. Is there a sense to make a connector for this web UI? Have you any use cases of this?
I'm not sure I understand your question. The existing, production-level Spotify web player UI at play.spotify.com also has a built-in scrobbler, yet this extension (which I'm extremely thankful for) also provides a connector for play.spotify.com.

In addition, I poked around my account settings in the new web UI and I have yet to find the option to link it with my lastfm account for scrobbling. It does not appear that the scrobbling feature has made it to the current version of the new UI: I don't know if they will include it in the final version if/when they release this to the general public.

Oops, sorry I posted with my other account I use for work :)
I'll just post it again with the account I used to start this thread for consistency.

So, as I understood, this is just an UI for Spotify Desktop app, right?
This is a new web player UI. I don't know if you have been selected for testing this new UI, but it looks like they are doing A/B testing on the new UI currently:
https://community.spotify.com/t5/Other-Partners-Windows-Phone-etc/New-Web-Player-is-garbage/td-p/1559947

Here is a longer thread: https://community.spotify.com/t5/forums/v3_1/forumtopicpage/board-id/001/page/1/thread-id/172874

The Spotify app is able to scrobble tracks by itself. Is there a sense to make a connector for this web UI? Have you any use cases of this?
I'm not sure I understand your question. The existing, production-level Spotify web player UI at play.spotify.com also has a built-in scrobbler, yet this extension (which I'm extremely thankful for) also provides a connector for play.spotify.com.

In addition, I poked around my account settings in the new web UI and I have yet to find the option to link it with my lastfm account for scrobbling. It does not appear that the scrobbling feature has made it to the current version of the new UI: I don't know if they will include it in the final version if/when they release this to the general public.

This is a new web player UI. I don't know if you have been selected for testing this new UI, but it looks like they are doing A/B testing on the new UI currently

I didn't select anything, I just opened open.spotify.com and it asks to install desktop application. After installing and starting of it this website was just like web UI for desktop player. That means it doesn't play songs without desktop player. This website looks like on your screenshot, so I decided it's what are you talking about.

I'm not sure I understand your question. The existing, production-level Spotify web player UI at play.spotify.com also has a built-in scrobbler, yet this extension (which I'm extremely thankful for) also provides a connector for play.spotify.com.

Seems like we're talking about different things. That question makes no sense, if open.spotify.com is web player, and not web UI.
Is anything I should to do to enable the new design?

Wow, it suddenly started to show me new web player on open.spotify.com.

I was about to reply, but I guess you found the answer to your question :)
I can take a crack at implementing this if you don't mind it taking some time as I'm new to developing extensions.

I added an experimental support of new website at https://github.com/alexesprit/web-scrobbler/archive/open-spotify.zip. Should work fine. Could you check that?

Testing it right now. It knows that a song is playing but is not able to figure out the song. Looks like track.getInfo is returning 400 Bad Request for some reason.
Ah I see. The artist and track query parameters are switched. Flip those two and it should be good.

I flipped the two selectors in your code and now it's working.
Thanks for the quick implementation!!!!

Thanks for note. I've already fixed that bug. I also fixed an issue when multiple artists were recognized incorrectly (e.g. 'Blood CommandSavant' instead of 'Blood Command, Savant').
I updated the 'open-spotify' branch; the update is available at the same URL.

:+1:
Looks good on my end. Looking forward to the next release!


Feature Name
AUTHENTICATION
Feature Description
THERE SHOULD BE AUTHENTICATION FEATURE SO THAT BY THIS USER CAN GET COURSES,AND EXPLORE OTHER FEATURES OF WEBSITE..
Motivation
THERE SHOULD BE AUTHENTICATION FEATURE SO THAT BY THIS USER CAN GET COURSES,AND EXPLORE OTHER FEATURES OF WEBSITE..
Implementation Suggestions (Optional)
WE CAN USE GOOGLE AUTHENTICATION..
Feature Type
AUTHENTICATION
Does this feature require additional resources?

[X] Additional Documentation
[X] Third-Party Libraries
[X] UI/UX Design
[ ] Other

References (Optional)
GOOGLE AUTH.

@ajay-dhangar Requested to assign me with level 3/4 as it contains backend as well as fronted designing..

@ajay-dhangar Which data base should i use ? tell me based on your requirement...

@ajay-dhangar Please tell me so that i can implement accordingly...

@ajay-dhangar Please tell me so that i can implement accordingly...


Rename V2 package tab IDs

analysis tab's name (score) was not following the same pattern
pub-tab-* was hard to read, -*-tab- is better, and seems to be unique enough that we won't have name collisions from user-provided content.

Seems kind of weird, they start with a dash, but ok.


example of test working with dynamic proxy

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
API changes are not detected in this pull request.


Update Azure-Vote sample with new billing model using pod release id label

Update Azure-Vote sample with new billing model using pod release id label

removed redundant billing label from deployment spec
removed redundant billingIdentifier from values.yaml
added new label 'azure-extesnsion-usage-release-identifier' on pod spec

Before this is merged, should we also have the doc ready?

PM/doc-team is working the doc changes. To unblock partners, we plan to share offline docs. We can go ahead and merge this.

In reply to:<PHONE_NUMBER>


docs: added sqlite dependency to docs (invenio-db)

Signed-off-by: Krzysztof Nowak<EMAIL_ADDRESS>

@lnielsen @slint can you merge this?


Ensure correct flows use enhanced bank upload journey

Before, the Providers::Means::IdentifyTypesofIncomesController redirected all
requests to the new Providers::Means::RegularIncomesController if the enhanced
bank upload setting was enabled.
Initially this seemed reasonable, however it was based on some incorrect
assumptions.
When the enhanced bank upload setting is enabled, only applications that are
using the bank upload journey should use the new flow.
This change seeks to redress the incorrect flows and ensure that only
non-passported applications that are using the bank statement upload journey use
the new how much, how often (enhanced bank upload) flow.
User scenarios that should lead to the enhanced bank upload journey
Note: In each scenario the setting is enabled and the application is uploading bank
statements. If either of those conditions aren't met, the user will be directed to the
existing :identify_types_of_income route.

Provider uploads bank statements, applicant is not employed
Client has completed means, applicant is not employed
Provider enters employment incomes
Provider enters full employment details

I noticed this when doing some other work re-structuring feature tests for the new journey.
To resolve the issue, I looked for every instance where :identify_types_of_income is returned as a flow step and replaced it with this conditional:
if application.using_enhanced_bank_upload_journey?
  :regular_incomes
else
  :identify_types_of_incomes
end

Specifically, I'd like feedback on whether these changes to the flow make sense.
Are there any other scenarios we need to account for, or are there any scenarios where this change doesn't make sense?


new problem with velocity

java.lang.NullPointerException: Cannot invoke "com.google.gson.JsonElement.getAsString()" because the return value of "com.google.gson.JsonObject.get(String)" is null
        at com.namelessmc.plugin.lib.nameless-api.Website.<init>(Website.java:45) ~[?:?]
        at com.namelessmc.plugin.lib.nameless-api.NamelessAPI.getWebsite(NamelessAPI.java:83) ~[?:?]
        at com.namelessmc.plugin.common.ApiProvider.api(ApiProvider.java:94) ~[?:?]
        at com.velocitypowered.proxy.scheduler.VelocityScheduler$VelocityTask.lambda$run$1(VelocityScheduler.java:203) ~[velocity.jar:3.1.2-SNAPSHOT (git-3ae93875-b138)]
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136) ~[?:?]
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635) ~[?:?]
        at java.lang.Thread.run(Thread.java:833) [?:?]
>

Originally posted by @Khikki in https://github.com/NamelessMC/Nameless-Plugin/issues/254#issuecomment-1126657556

Your website is not sending the "locale" in the json response, this was added a couple weeks ago. Please note that the development version of the plugin is only compatible with the latest development version of NamelessMC

Can you tell me how can I install the latest version of nameless?

Development releases can be downloaded from https://ci.rkslot.nl/view/NamelessMC/job/Nameless/ these should not be used for anything other than testing purposes. Upgrades are not supported, frequent reinstallations are required

I tried to install my theme and module in the new version for test, otherwise localization will be called?
https://debug.namelessmc.com/s4lylEhoPK

Templates and modules made for pr12 are very likely incompatible with the development version. In this case it looks like it is missing files for the new language system

I'll close this issue since it seems to be resolved


Update README.md

Have some typo fixes (e.g. uppercase Swift in some cases). Parked this in the internal repo until I get access here.


Update Layout and styles

This PR updates the overall style to have better extensibility.
Honestly, I wanted to drop in Tailwind but the project was giving me far too many errors. when trying to do so. Instead I opted for some plain old CSS.
This isn't anything amazing but I think bring some clarity and uniformity to the elements.

Note that in this PR I modified a few components to extend from react component using a cleaner syntax of:
export default class Sidebar extends React.Component {

This made it easier to create component based methods.

Also, on my local there are quite a few warnings from the maps library. I don't believe my changes caused those warnings but you might want to double check.


scrollIntoView fails on document.body

I realized the scripts throws an error when calling document.body.scrollIntoView({ behavior: 'smooth' }) after investigating https://github.com/iamdustan/smoothscroll/issues/28 issue.
We also need to investigate the expected behavior of that function call, since Chrome does nothing while Firefox scrolls to top.

Looks like Chrome, Firefox, Safari, and Edge all behave as expected when using document.body and go to the top of the page: https://s.codepen.io/graygilmore/debug/RJgYRx
Only Chrome and Firefox respect the smooth behaviour still, though.


[Feature request] Support .png to gfx files

Add .png support on par with .dds and .tga (Used by HOI4)
The .png file format HOI4 can read and output, I think that it is worth adding support for viewing .png files

Really? I didn't see any doc that said that it support .png files. Let me have a try about it.

Yes, it works. How does the minium work in my project and so do the guys from the CW

How are you doing?

Hi @Sepera-okeq, I'll add png support in the next release.

.png had been supported in 0.4.5. Could you have a try?

does not work

if anything, I have already restarted and checked the version

It was frameAnimatedSpriteType that was not supported by the extension. I
can add support for it without animation.
sepera_okeq<EMAIL_ADDRESS>于2021年2月21日周日 下午5:13写道：

[image: image]
https://user-images.githubusercontent.com/54331959/108620584-7ab2c100-7446-11eb-9a77-ea1b40dbae81.png
if anything, I have already restarted and checked the version
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/herbix/hoi4modutilities/issues/24#issuecomment-782824745,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABWICNMT6APW6C2XF7XKLBTTADFDLANCNFSM4XJUHBVQ
.

--
Chaofan Yang
Tel: (+86)15510769284

Oops. If we talk about the usual SpriteType, then yes, it works.

You can just make a separate setting for the animation. To either display just as a picture or animation :)

0.4.6 supported animated sprite type.


Krippendorff Library Returning Nan

Sometimes during the krippendorff library is outputting nan values when there should be a numeric value.
It's noticed in the evidence schema, on article 3be14d67e2d88964904dcbe7df176bb81dacfc76a6f2e4ec66b45681c86c9301 for question 1 answer 1, and question 2 answer 3.
After a brief review of the literature on Krippendorff's alpha for unitizing agreement, I don't think this is a feature of that algorithm.
At this point I'm not sure what is causing it, the library we're using notes that they compute it slightly differently than was intended in order to improve runtime.  It could be an artifact of this, but I'm not sure.
Not 100% sure what the best route forward is, might be ideal to just create our own implementation of krippendorff's alpha.
Right now the systems behavior is to ignore the nan scores, and to output an agreement score as if there were no unitizations for the question.  This could lead to final point values being incorrect.

Krippendorff is removed.  Algorithm team will work on a replacement method this semester.


chore(eks): update eks instance type list with max pods

I'm unable to spin up EKS worker nodes with T3 instances. I noticed the MAX_PODS instance list is way behind.
The information in this list is directly copied from: https://github.com/awslabs/amazon-eks-ami/blob/master/files/eni-max-pods.txt
Pull Request Checklist

[ ] Testing

Unit test added (prefer not to modify an existing test, otherwise, it's probably a breaking change)
CLI change?: coordinate update of integration tests with team
cdk-init template change?: coordinated update of integration tests with team

[ ] Docs

jsdocs: All public APIs documented
README: README and/or documentation topic updated
Design: For significant features, design document added to design folder

[ ] Title and Description

Change type: title prefixed with fix, feat and module name in parens, which will appear in changelog
Title: use lower-case and doesn't end with a period
Breaking?: last paragraph: "BREAKING CHANGE: <describe what changed + link for details>"
Issues: Indicate issues fixed via: "Fixes #xxx" or "Closes #xxx"

[ ] Sensitive Modules (requires 2 PR approvers)

IAM Policy Document (in @aws-cdk/aws-iam)
EC2 Security Groups and ACLs (in @aws-cdk/aws-ec2)
Grant APIs (only if not based on official documentation with a reference)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license.

Just noticed the EKS module needs some more updates. Version 1.11 is listed as the latest version, and the AMI IDs for the EKS AMIs are out of date.
Also, Kubernetes 1.13 was released on EKS yesterday. The UI says that 1.12 is "default". The CDK defaults to a "LATEST_KUBERNETES_VERSION" variable: should that be changed to a "DEFAULT_KUBERNETES_VERSION" variable instead?
I would also suggest to remove the mention of version 1.10 (and maybe even 1.11) from the CDK. Thoughts?
I can append this PR with the changes.


helio-workstation: 3.3 -> 3.4

Motivation for this change
Things done

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

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

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

Result of nixpkgs-review pr 117453 at c28c61b7 run on x86_64-linux 1

1 package built successfully:

helio-workstation

1 suggestion:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/applications/audio/helio-workstation/default.nix:31:3:
   |
31 |   installPhase = ''
   |   ^


Fix flaky integration test for max_runtime_exceeded

Closes #299

@dposada and I were discussing what to do about this test this morning. We concluded that making the test a little more robust/forgiving would be better than marking it as explicit.

@shamsimam - Feedback addressed.

@dposada - Feedback addressed.

@dposada - I added the comment explaining why we have three consecutive wait calls.


Update hooks version for PureScript v0.14

What does this pull request do?
This PR updates the hooks-based version of the package to work with PureScript v0.14 and Halogen 6. So far I managed to get the basic examples to build but had to do some minor changes for halogen-select here.
Where should the reviewer start?
Quick synopsis for changes:

Applied general updates for Halogen 6
Unified proxy types to use just Proxy
Removed use of HookEquals

How should this be manually tested?
There shouldn't be any major changes in terms of behaviour.
Other Notes:
External stuff:

[x] halogen-storybook in package sets
[ ] halogen-select

Thanks for the PR, @PureFunctor! I can have a thorough look at this next week and merge it in (it pretty much looks 👍).
I'm not entirely happy with the Hooks branch of Formless, and I don't have a concrete plan to release it. Please feel free to the branch if you're relying on it in a project, but I can't promise that it won't change drastically in the future!


fix: Gemfile & Gemfile.lock to reduce vulnerabilities

The following vulnerabilities are fixed with an upgrade:

https://snyk.io/vuln/SNYK-RUBY-NOKOGIRI-459107

review required

Merge

Close


integrate the vision document

r? @tmandry

Hey all! Thanks for the feedback. I've got a bunch of revisions planned, both to the overall structure of this section but also in response to specific comments. For the time being, I'm going to close this PR while I work on that and re-open, hopefully today/tomorrow.

here are some notes I took on the major points people raised:

examples around await and JS didn't quite get it right

if you have foo.await on each line in JS, it blocks too
people might use Promise.all for example in JS
it's more idiomatic to use join than spawn

to avoid excess threads, I presume, under the assumption that I will get enough parallelization by having many connections

be specific about concurrency, parallel, asynchrony
not all things in async have counterparts in sync

this was intended, but not made explicit in the tenet

can/should we integrate "easy to switch runtimes" as a tenet?
do we really want a lint to help people identify potential concurrency?
cancellation for task spawning is a bit tricky
we should identify interation into glib, QT, android, windows explicitly as goals
how much do we focus on the initial learning experience and initial intuitions?
to what extent is this vision doc targeting the lang vs the rust ecosystem at large

the wording implies "standard library", is that what is meant?

ordering of tenets: is it more important to give base capabilities, or to make async like sync, if they are in tension?

to avoid excess threads, I presume, under the assumption that I will get enough parallelization by having many connections

The main reason I see to prefer join! is that, besides being simpler, it also has a better default panic and cancellation behavior for the kind of situation you used in the example.

@Darksonn thanks! I added that.


suspicious-package: Why a qlplugin stanza pointing to plugin inside app bundle?

In #19558, suspicious-package.rb was updated to reference the new app, but the qlplugin stanza was changed to reference the Quick Look plugin within the app bundle:
qlplugin 'Suspicious Package.app/Contents/Library/QuickLook/Suspicious Package.qlgenerator'

The resulting symlink in ~/Library/QuickLook is redundant (quicklookd will find the plugin embedded in the app bundle, as is standard). More importantly, it causes Suspicious Package.app to generate a confusing alert about cleaning up an old version of the standalone plugin. (Since Suspicious Package was changed from a standalone plugin to an app, it tries to be helpful and offer to clean up the old plugin, since the bundled one is newer and will be used by QL anyway. However, it doesn't expect to find a symlink pointing back at its own bundle, so doesn't handle this situation properly. As a result, it tells the user that the "old" version is the same as the new version, and "cleanup" just trashes the symlink -- not fatal but confusing.)
Yes, Suspicious Package should be smarter about this, but what purpose does this separate symlink even serve?

Pinging @russellhancox as to why.
If there’s no reason, we’ll remove that line.

I didn't realize quicklookd would find the one inside the app bundle automatically, you can remove that line. The cask had been failing to install because the referenced qlplugin didn't exist to create a symlink from.

Done. Thank you @russellhancox for the fast reply (and original fix) and @rsaldinger for pointing out the mistake.

Awesome. Thanks for the speedy fix!


🚀<EMAIL_ADDRESS>Update all dependencies

Deploy preview for blog-thibmaekelbergh failed.
Built with commit 98381d66ad31defcda9c19800b694341e4499ab2
https://app.netlify.com/sites/blog-thibmaekelbergh/deploys/5b01b6ebe39e7c2dcefc73a4


Resolve DeprecationWarnings triggered by tests

Our tests are hitting a lot of deprecation warnings now, especially in (but not limited to) numpy. This issue can be where we inventory, classify, and discuss resolving them.

From merging of #207 on ubuntu-latest/Python3.10 I see:

bifrost/DataType.py:51: UserWarning: The NumPy module was reloaded (imported a second time). This can in some cases result in small but subtle issues and is discouraged.
test/test_block.py:479: DeprecationWarning: Conversion of an array with ndim > 0 to a scalar is deprecated, and will error in future. Ensure you extract a single element from your array before performing this operation. (Deprecated NumPy 1.25.)
bifrost/block.py:303: DeprecationWarning: product is deprecated as of NumPy 1.25.0, and will be removed in NumPy 2.0. Please use prod instead.
bifrost/sigproc2.py", line 366, in read -> self.buf.resize(requested_nbyte)
ValueError: cannot resize an array that references or is referenced by another array in this way.  Use the np.resize function or refcheck=False NB:  This one is an error but it doesn't cause problems.
lint/run.py:219: DeprecationWarning: do_exit is deprecated and it is going to be removed in a future version.
bifrost/ndarray.py:222: DeprecationWarning: NumPy will stop allowing conversion of out-of-bound Python integers to integer arrays.  The conversion of 152 to int8 will fail in the future.

For bifrost/block.py:303 the line number is misleading.  It actually looks like the problematic lines are inSplitterBlock.load_settings().

For test/test_block.py:479 I get that next(span_gen).data is a 2D array, making .data[0] a 1D array.  Maybe .data.item() for this one?

I was looking through our oh-so-many branches this morning and I found numpy_deprecated_dtypes which might be relevant here.  I'm not seeing any of these warnings in CI though.

I'm looking at bifrost/sigproc2.py:366 to see how self.buf is used.  For data with nbit >=8 it isn't; the whole self.buf.resize() thing is only for sub-byte sized data.  In that case self.buf directly becomes the data which SigprocFile.read() returns.  In light of that maybe a conversion of something like
self.buf = np.resize(self.buf, requested_nbyte)

and
self.buf = np.resize(self.buf, nbyte)

would be the best fix.

There is also #219 now.

With #219 merged it looks like we are down to the numpy.resize thing and the occasional "unrecognized character" that comes from somewhere.


Problem with $XXXX hexadecimal numbers

Hello
First of all, my excuses if the issue isn't at all related to monokai extended. I guess it may be as I've been given the advice to use monokai extended to work with markdown. If it's not, apologies.
I'm writing .md documents and I've found a problem whenever I enter a hexadecimal number in the $XXXX format, which is a pretty common practice specially if you are referencing assembly listings (which I am).  Syntax is off from that onwards, I don't know if there's a workaround for this?
Something like "this is a block of text which references something which is located at address $8000 in the CPU address space". Everything goes haywire from the "$" onwards.
Screenshot

sorry for the late reply. did you ever find a solution to this? I was not able to reproduce, I tried a bunch of different configurations with this plugin and others and didn't see this happen. My guess is that something before that line is actually the cause of the problem, but it's only manifesting there.
I'm going to go ahead and close, since it seems like this is something unrelated to this lib, but please feel free to reopen if you have any more info.


PowerShell 7 compatibility

In PowerShell 7, Import-Module Fasdr results in:
InvalidOperation: <User profile path>\Documents\PowerShell\Modules\Fasdr\1.0.135\Fasdr.psm1:288
Line |
 288 |          if ((test-path $path) -and $global:fasdrDatabase.AddEntry($fi …
     |              ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | You cannot call a method on a null-valued expression.

Are there any plans to support 7 or any idea what needs to be done?

It's been awhile since I did development in that repo. Let me see what I can do.


Add usage documentation on readme

I've had to figure out how to use this small module and could not find any information on the website.
So I took the liberty to add a small Usage section to the readme.

Codecov Report

Merging #19 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #19   +/-   ##
=======================================
  Coverage   86.09%   86.09%
=======================================
  Files          18       18
  Lines         266      266
  Branches        9        9
=======================================
  Hits          229      229
  Misses         37       37

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 93edf84...04cd877. Read the comment docs.


HDDS-7161. Make Checksum.int2ByteString() zero-copy

What changes were proposed in this pull request?
Remove expensive heap memory copy; replace with zero copy wrapper.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-7161
How was this patch tested?
Covered by existing UT.

Thanks @jojochuang for the patch, @kerneltime for review.


Run separate API requests async to reduce latency

If we can run multiple queries simultaneously then each /search request will only be as slow as the slowest individual child API call.

Closed by #9


Fix styling

Upgrade Semantic UI version
Make scrollbar appear always

Make marker info window not scroll/cut off on Safari
Switch 'Upload' and 'Reset' order

Added in logo:

Right-most side of nav only on desktop
At bottom of 'About' page

@riyer15


Add back prompt for portable installer

[x] Have you signed the Contributor License Agreement?
[x] Are you working against an Issue?

Resolves #353

Add back the prompt to ask for portable installers and set the InstallerType to portable accordingly.
Verified by running unit tests locally and with the InstallerUrls mentioned in the issue

Microsoft Reviewers: Open in CodeFlow

/azp run

/azp run


fix: UI_REACT: Nested DataListItem in MappingField causes console warning

Fixes: #1840

I ran yarn format to correct. Not sure if I understand the result, but...


Use Cases

I think we should have either a repo, or a subdirectory of a repo, for people to submit use cases.
I will use the term use cases in the general English usage of the words (ie a case where something is used) but the software development purists might claim these are scenarios not formal use cases (note I didn't use word formal in what I proposed).
IMHO people shouldn't refute other people's use cases (just because sFractal doesn't do something doesn't mean someone else doesn't do it).  Conversely, people can't submit other people's use cases (eg I at sFractal can't say 'at Redhat they do .... or as happens more often 'I think someone might ...'). Ie just speak for yourself.
If this gets contentious (it usually does as we tend to talk past each other assuming other people use our context not theirs), I found it useful to have a subdir for 'agreed to general usecases' and a subdir for 'contributor use cases'. We start with everyone submitting their own use cases into their own subdir of the contributor use cases (eg I'd put them in usecases/contrbutor/sFractal). If everyone agrees a particular sfractal use case is 'general enough', then it gets copied (maybe edited first) into usecases/general. Usually we don't need the general use cases and just develop the spec from the pile of contributor use cases. But having documented use cases does allow all of us to see the different perspectives.
This tends to be most useful in contrasting situations. One is the commercial closed source vendor vs the MIT license open source github library (and I realize there are many other variants than these two). Another is between hard-to-update-IoT devices vs classic server app vs cloud whatever-as-a-service. Delineating the use cases helps us recognize when person A was talking one situation applicable to their environment and person B was talking about a different situation applicable to their environment.
The other advantage of the 'everybody in their own subdir for use cases' and 'don't bother about the general use casses' is you don't spend months arguing about them.

Hi @sparrell,
I wholeheartedly agree with your suggestion to establish a repository, or a subdirectory within an existing repo, for the submission of use cases. This approach not only streamlines the collection of various scenarios but also encourages a broader understanding of how diverse environments and requirements can influence the application of our work.
Your point about the non-refutation of use cases is well-taken. It's important that our platform remains open to how individuals and organizations uniquely utilize the software, rather than limiting our scope to what we currently support or envision. This open-mindedness is crucial for innovation and adaptability.
The structure you propose — with a dedicated subdirectory for 'contributor use cases' and a potential 'agreed to general use cases' — is a pragmatic solution to avoid contention. It encourages contributors to freely express their unique scenarios, while also providing a path to consolidate widely applicable use cases.
I'm in favor of moving forward with this proposal and would be interested in discussing the next steps with the TC to implement this system effectively.
Now that we have the new TC formed and the TC GitHub repository. I will move this suggestion to the working items /issues there.


asm-blocks parser

The parser should produce ASTs for asm-blocks. This will ensure we are in control of error messages and also we can later implement a type-checker for the asm-blocks.

Wrote a significant amount of Ohm parser code and parts of later stage hacks to support defining new words, including the active words, which affect the parsing in a context-sensitive way — the active words take the following word as an argument to them!
But then I realized, that:

Shadowing is allowed (and occurs tacitly/silently), meaning that even built-in words can be re-defined while doing ANY parsing, so my updates to grammar.ohm itself are wasted on that.
forget can remove the words. It can even forget forget!

I have a workaround with the simple recursive-descent parser and a dictionary to keep track of, but for that I'll need the upcoming day to complete this issue. Otherwise we're left with just a slight expansion of possibilities of the current parser and a better recognition of built-in words in Fift-asm.


[Fix-7857] The user won't be allowed to disable their own account.

Purpose of the pull request
This PR will close #7857 .

Brief change log
It's risky to disable some users' account by themselves. So rectify the codes not to allow user disable their own account.
Verify this pull request
This change added tests and can be verified as follows:

Manually verified the change by testing locally.

Codecov Report

Merging #7891 (5f5439e) into dev (6532215) will decrease coverage by 0.00%.
The diff coverage is 40.00%.

@@             Coverage Diff              @@
##                dev    #7891      +/-   ##
============================================
- Coverage     40.98%   40.98%   -0.01%
+ Complexity     3694     3693       -1
============================================
  Files           637      637
  Lines         27053    27057       +4
  Branches       3072     3073       +1
============================================
  Hits          11089    11089
- Misses        14896    14898       +2
- Partials       1068     1070       +2

Impacted Files
Coverage Δ

...inscheduler/api/service/impl/UsersServiceImpl.java
75.67% <0.00%> (-0.48%)
:arrow_down:

.../org/apache/dolphinscheduler/api/enums/Status.java
100.00% <100.00%> (ø)

...er/master/dispatch/host/assign/RandomSelector.java
77.77% <0.00%> (-5.56%)
:arrow_down:

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 6532215...5f5439e. Read the comment docs.

When editing someone's own user profile, the component of disabling  user's state will never show up.


Following zgui's README.md results in an error (missing build.zig.zon)

I followed zgui's README.md file by copying the required libs into a new project (initialized by zig init-exe) and then adjusting a build.zig as demonstrated, and the only error I've got after trying to build was this:

$ zig build
no dependency named 'dawn_x86_64_linux_gnu' in '/home/dima/projects/zgui-test/build.zig.zon'. All packages used in build.zig must be declared in this file.
error: the following build command failed with exit code 1

I successfully resolved it by copying build.zig.zon from zig-gamedev project's root to my project root and then changing the name of the package there.
Not sure if this is the correct way to solve it, but if it is, I can create a PR with an updated README.md, mentioning this step.

Thanks for the info, please make a PR.


Configure the generic exception handler

needs a handler for the ApiExceptions
needs a general handler for all uncaught exception

PR #9 is closed


Refactoring Maven manifest reading

There are two places we read Maven pom.xml:

Transitive scanning in internal/manifest
Guided remediation in internal/resolution/manifest

Both share the same logic to merge parents, so this PR consolidates the implementation in internal/manifest.
This PR also updates deps.dev dependencies to the latest version.

Codecov Report
Attention: Patch coverage is 83.78378% with 6 lines in your changes missing coverage. Please review.

Project coverage is 66.23%. Comparing base (fc67a78) to head (073ec19).
Report is 1 commits behind head on main.

Files
Patch %
Lines

internal/resolution/manifest/maven.go
77.77%
0 Missing and 4 partials :warning:

internal/manifest/maven.go
88.23%
0 Missing and 2 partials :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1159      +/-   ##
==========================================
+ Coverage   65.70%   66.23%   +0.52%
==========================================
  Files         164      164
  Lines       13682    10582    -3100
==========================================
- Hits         8990     7009    -1981
+ Misses       4206     3090    -1116
+ Partials      486      483       -3

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


Adding MaxMind's GeoIP2 Precision Service API

Connected to #594
I know that geocoder currently has a geoip2 lookup defined which uses Hive::Geoip2

though I believe this is the lite version using the free database?
should I be adding this functionality to :geoip2 directly and have some logic that understands to use the Precision Service API?

I ran the test cases, but locally bundle exec rake test seems to indicate that the test_helper's version of MaxmindGeoip2 might not actually be used

Do I have something configured wrong for the tests?

Thanks for this. Let's address these two items and then revisit the test failures:

You don't need to modify Lookup::Base#make_api_request. If you look at the top of that method you'll see the username and password being extracted from the parsed URI. Just generate a query URL with the username and password included (eg: http://user:pass@geoip.maxmind.com/...).
You don't need to add :basic_auth to the base configuration hash. Each lookup will have its own credentials so the developer should do that configuration for this lookup only, as you do correctly in test_helper.rb (no Geocoder code modification needed).

@alexreisner
About not modifying Lookup::Base#make_api_request - I wasn't too sure about the whole basic_auth verses proxy. I thought they were different. Or is it that the proxy can be used instead of using basic authentication?

Hmmm...I believe that syntax is just how basic auth works. I don't think it implies a proxy.

For the format that you mentioned<EMAIL_ADDRESS>Lookup::Base#http_client parses it using URI and creates a Net::HTTP::Proxy which is then used to run the get
The GeoIP2 Precision Web Services points to http://en.wikipedia.org/wiki/Basic_access_authentication for more info about basic auth.

Look at line 104. It only creates a proxy if :http_proxy or :https_proxy is configured.

Line 269 is where user/pass are extracted from the URI. On lines 274 and 278, in the http_client.start block, you should be using client, not http_client (which may be the source of the test failures), but again, I don't think this code should be modified.

@alexreisner Ah okay I was slightly confused.
I did a simple test with Net::HTTP using http://httpbin.org/
$ irb
2.2.0 :001 > require 'net/http'
 => true
2.2.0 :003 > uri = URI.parse("http://httpbin.org/basic-auth/user1/passwd2")
 => #<URI::HTTP http://httpbin.org/basic-auth/user1/passwd2>
2.2.0 :017 > args = [uri.host, uri.port, nil, nil, "user1", "passwd2"]
 => ["httpbin.org", 80, nil, nil, "user1", "passwd2"]
2.2.0 :018 > client = Net::HTTP.start(*args, {})
 => #<Net::HTTP httpbin.org:80 open=true>
2.2.0 :021 > req = Net::HTTP::Get.new(uri.request_uri)
 => #<Net::HTTP::Get GET>
2.2.0 :022 > client.request(req)
 => #<Net::HTTPUnauthorized 401 UNAUTHORIZED readbody=true>
2.2.0 :023 > req.basic_auth("user1", "passwd2")
 => ["Basic dXNlcjE6cGFzc3dkMg=="]
2.2.0 :024 > client.request(req)
 => #<Net::HTTPOK 200 OK readbody=true>

It seems that basic authentication with net/http can only be done through the request.basic_auth method, and not the client args.

Okay, thanks, that does seem like a good proof. Just be sure to use client instead of http_client within the start block, or get rid of the start block if it's not necessary. Also remove the args-related lines if they're not necessary. But make sure to preserve SSL support!

@alexreisner Yes thanks I have made sure to use the client instead of http_client within that start block.
I found out the issue I was having with failing tests was because the query_url that I had for maxmind_geoip2 was wrong (i.e. did not generate a unique url so that the caching could work properly). Thus the same result was being grabbed for each call.
I looked back at the documentation for GeoIP2 and the url should be:
https://geoip.maxmind.com/geoip/v2.1/{service}/{ip_address}
where in service is one of city, country, or insights
I have made those changes to the url in maxmind_geoip2.
I have also manually tested geocoder a little to ensure the the basic_auth addition to make_api_request did not break everything.
Thus-MacBook-Air:geocoder trangpham$ gem build geocoder.gemspec
  Successfully built RubyGem
  Name: geocoder
  Version: 1.2.7
  File: geocoder-1.2.7.gem
Thus-MacBook-Air:geocoder trangpham$ gem install ./geocoder-1.2.7.gem
Successfully installed geocoder-1.2.7
Parsing documentation for geocoder-1.2.7
Done installing documentation for geocoder after 1 seconds
1 gem installed
Thus-MacBook-Air:geocoder trangpham$ irb
2.1.2 :001 > require 'geocoder'
 => true
2.1.2 :002 > Geocoder.configure(timeout: 10, lookup: :bing, units: :km, api_key: "MY_API_KEY_IS_HIDDEN")
 => {:timeout=>10, :lookup=>:bing, :ip_lookup=>:freegeoip, :language=>:en, :http_headers=>{}, :use_https=>false, :http_proxy=>nil, :https_proxy=>nil, :api_key=>"MY_API_KEY_IS_HIDDEN", :cache=>nil, :cache_prefix=>"geocoder:", :basic_auth=>{}, :always_raise=>[], :units=>:km, :distances=>:linear}
2.1.2 :003 > Geocoder.search("San Francisco, CA")
 => [#<Geocoder::Result::Bing:0x007faeb567b568 @data={"__type"=>"Location:http://schemas.microsoft.com/search/local/ws/rest/v1", "bbox"=>[37.64891052246094, -122.66365814208984, 37.92567443847656, -122.18305206298828], "name"=>"San Francisco, CA", "point"=>{"type"=>"Point", "coordinates"=>[37.77711868286133, -122.41963958740234]}, "address"=>{"adminDistrict"=>"CA", "adminDistrict2"=>"San Francisco Co.", "countryRegion"=>"United States", "formattedAddress"=>"San Francisco, CA", "locality"=>"San Francisco"}, "confidence"=>"High", "entityType"=>"PopulatedPlace", "geocodePoints"=>[{"type"=>"Point", "coordinates"=>[37.77711868286133, -122.41963958740234], "calculationMethod"=>"Rooftop", "usageTypes"=>["Display"]}], "matchCodes"=>["Good"]}, @cache_hit=nil>, #<Geocoder::Result::Bing:0x007faeb567b0b8 @data={"__type"=>"Location:http://schemas.microsoft.com/search/local/ws/rest/v1", "bbox"=>[37.78520584106445, -122.40876007080078, 37.80015563964844, -122.39292907714844], "name"=>"Financial District, CA", "point"=>{"type"=>"Point", "coordinates"=>[37.79383087158203, -122.40196990966797]}, "address"=>{"adminDistrict"=>"CA", "adminDistrict2"=>"San Francisco Co.", "countryRegion"=>"United States", "formattedAddress"=>"Financial District, CA", "locality"=>"Financial District"}, "confidence"=>"High", "entityType"=>"Neighborhood", "geocodePoints"=>[{"type"=>"Point", "coordinates"=>[37.79383087158203, -122.40196990966797], "calculationMethod"=>"Rooftop", "usageTypes"=>["Display"]}], "matchCodes"=>["Good"]}, @cache_hit=nil>]

Thanks @alexreisner

Thank you! I merged this and made a few minor changes. This is the most substantive change: 7331bd7d.
Also, see #801.


jmp & jmpif instructions on wiki

you forgot to add jmp and jmpif isntructions (along with jump labels)

yea


Allow possibilities disabled hologram and set SIGN.

Hello
When it's possible to add SIGN on chest dans disabled hologram to config ?
Because 10 shop its good but when we take +100 shop and impossible set a chest on a chest, its not good for player.

There will never be sign shops in this plugin.


Unzip with overwrite_files_names not working

    {
        "name": "SumatraPDF",
        "url": "https://www.sumatrapdfreader.org/dl/rel/3.4.6/SumatraPDF-3.4.6-64.zip",
        "decompress_flat": true,
        "overwrite_files_names": {
            "^SumatraPDF.*":"SumatraPDF.exe"
        },
        "version": "3.4.6"
    },

Regex must exclude the origin zip file, because renaming is used for downloading and unzip.
    {
        "name": "SumatraPDF",
        "url": "https://www.sumatrapdfreader.org/dl/rel/3.4.6/SumatraPDF-3.4.6-64.zip",
        "decompress_flat": true,
        "overwrite_files_names": {
            "^SumatraPDF.*.exe":"SumatraPDF.exe"
        },
        "version": "3.4.6"
    },


Add the model version tag to any place where the model version is currently displayed

Also, update the branch selector widget to use a material selector, and present branches / versions separately.
Before:

After:

Depends on https://github.com/MauroDataMapper/mdm-core/pull/448 and https://github.com/MauroDataMapper/mdm-resources/pull/110


ActionType could be enum of TypeScript w/o using keymirror

For now we are just using plain json. Types can be known statically

Done by #84


It should be clear IHostedService.StartAsync() should be short lived.

Under BackgroundService  is states StartAsync should be limited to short running tasks, however this comment also applies to IHostedService .

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

ID: c3460b0d-9c60-3a06-ac63-18cf990e53c3
Version Independent ID: b2dece1f-15b8-03af-6d52-7cc070432b4b
Content: Background tasks with hosted services in ASP.NET Core
Content Source: aspnetcore/fundamentals/host/hosted-services.md
Product: aspnet-core
Technology: aspnetcore-fundamentals
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

@gklittlejohn  are you saying it should change to the following?
IHostedService.StartAsync should be limited to short running tasks because hosted services are run sequentially, ...
Any chance you could PR this? At the top right of the article, select the Edit pen icon:

Select the pen icon again. Save.

Yes, I'll do this when I have time.


[Bug]: withFileSystemBind does not mount any files

Kubedock Version
0.9.2
Host OS
MacOS
Host Arch
ARM
Environment:
Local Cluster with k3d:
Server Version: version.Info{Major:"1", Minor:"21", GitVersion:"v1.21.10+k3s1", GitCommit:"471f5eb3dbfeaee6b6dd6ed9ab4037c10cc39680", GitTreeState:"clean", BuildDate:"2022-02-24T23:38:19Z", GoVersion:"go1.16.10", Compiler:"gc", Platform:"linux/arm64"}
Gitlab CI/CD pipeline k8s cluster:
Server Version: version.Info{Major:"1", Minor:"21", GitVersion:"v1.21.9", GitCommit:"b631974d68ac5045e076c86a5c66fba6f128dc72", GitTreeState:"clean", BuildDate:"2022-01-19T17:45:53Z", GoVersion:"go1.16.12", Compiler:"gc", Platform:"linux/amd64"}
Steps to Reproduce:

Crate a new Container
Add a withFileSystemBind of any folder
Start the container

Expected:

Folder is successfully bound to container and files are available

Actual:

The folder was not mounted.

Additional Information:
The argument "--pre-archive" needs to be set for kubedock. Otherwise no copy/mount did work at all.
We tried mounting and copying files to the container with all available possibilities.
None of them is working on it's own. Meaning if you use any of the following methods
no file is ever mounted or copied.

.withFileSystemBind(...)
.withCopyToContainer(...)
.withCopyFileToContainer(...)
.withClasspathResourceMapping(...)

However when combining multiple of these Methods they partly work.
Whenever the method ".withFileSystemBind(...)" is present for the container, all
other copies work as expected. But the filesystembind itself does not. In this case
there is a config map as well as the mounted files present in the container.
Following shows the describe for the started alpinecontainer.
With all 4 methods called:
	Mounts:
		/config2 from pfiles (rw,path="fbc1ac88b427356b61892951faa93e54")
		/config3/redis.conf from pfiles (rw,path="c7f85a5709e2ec3a35488d25ef0bdde7")
		/config4/redis.conf from pfiles (rw,path="377f7586930fe9b2b041e64a12d5e76e")
		/var/run/secrets/kubernetes.io/serviceaccount from kube-api-access-r4f4t (ro)
	Volumes:
		pfiles:
			Type: ConfigMap (a volume populated by a ConfigMap)
			Name: e363d58d2cf8
			Optional:  false

The same output without the .withFileSystemBind BUT still we the other 3 copies (no volume present afterwards):
	Mounts:
		/var/run/secrets/kubernetes.io/serviceaccount from kube-api-access-jn4c8 (ro)

Used example test:
@Test
@SuppressWarnings("unchecked")
void shouldBindDirectory() {
    String configPath = Paths.get("").toAbsolutePath() + "/build/it-config/data";
    File file = new File(configPath + "/redis.conf");
    GenericContainer test = new GenericContainer(DockerImageName.parse("alpine:3.15"))
        .withLogConsumer(new Slf4jLogConsumer(getLogger(this.getClass().getName())))
        .withFileSystemBind(configPath, "/config", BindMode.READ_ONLY)
        .withCopyToContainer(Transferable.of(file.getAbsolutePath()), "/config2/redis.conf")
        .withCopyFileToContainer(MountableFile.forHostPath(file.getAbsolutePath()), "/config3/redis.conf")
        .withClasspathResourceMapping("redis.conf", "/config4/redis.conf", BindMode.READ_ONLY)
        .withCommand("sh", "-c", "sleep 5 && echo started && tail -f /dev/null")
        .waitingFor(Wait.forLogMessage(".*started.*", 1).withStartupTimeout(Durations.ONE_MINUTE));
    test.start();
    try {
        String basePath = test.execInContainer("ls", "/").getStdout();
        log.info(basePath);
        String foo1 = test.execInContainer("ls", "/config").getStdout();
        String foo2 = test.execInContainer("ls", "/config2/").getStdout();
        String foo3 = test.execInContainer("ls", "/config3").getStdout();
        String foo4 = test.execInContainer("ls", "/config4").getStdout
        log.info("TEST withFileSystemBind: " + foo1);
        log.info("TEST withCopyToContainer: " + foo2);
        log.info("TEST withCopyFileToContainer: " + foo3);
        log.info("TEST withClasspathResourceMapping: " + foo4);
    } catch (IOException | InterruptedException e) {
        throw new RuntimeException(e);
    }
}

This does only occur when executing in gitlab-ci pipeline or for testing purposes in a local cluster.
Running it locally in docker works as expected.

I am trying to reproduce as well (using minikube), but I can't. Both with and without the --pre-archive all tests produce the expected output. Is the instance of kubedock able to directly access the files, or folders? What I have seen e.g. in similar issues, is that kubedock is running in a sidecar, but the actual folders that are to be bound are not available in that sidecar; maybe it is something similar?

Did some more extensive testing and the only way to reproduce is when the volumes which are to be bound, are not directly accessible by kubedock. The other working methods in your tests are methods that copy the files from the running test system via the rest api, and don't require direct access to the filesystem.
See also the tekton example here, where the source folder is also mounted in the sidecar, making the folders directly accessible for kubedock as well.

I will check out the example and provide you with more information if needed. Had no time yet to dig into.

I stepped through the kubedock code when called from testcontainers.net
I inspected the raw request contents and noticed that the mounts are defined in HostConfig.Mounts but the kubedock struct for HostConfig only supports Binds not Mounts: https://github.com/joyrex2001/kubedock/blob/8b1f7ce3d08386cbd4f20cdc13ca9c4119f9a766/internal/server/routes/docker/types.go#L39-L44
Not sure if this is also what testcontainers java does but I think kubedock should support Mounts since IIRC it's the more recent and feature-rich API?

I stepped through the kubedock code when called from testcontainers.net
I inspected the raw request contents and noticed that the mounts are defined in HostConfig.Mounts but the kubedock struct for HostConfig only supports Binds not Mounts: https://github.com/joyrex2001/kubedock/blob/8b1f7ce3d08386cbd4f20cdc13ca9c4119f9a766/internal/server/routes/docker/types.go#L39-L44
Not sure if this is also what testcontainers java does but I think kubedock should support Mounts since IIRC it's the more recent and feature-rich API?

I'm not sure if I have similar issue. But only happen with testcontainers .NET and not java or node. So something different on implementation.
The point is that expect my container (ryuk) to be created with sidecars when it's not. So it could be possible the socket bind created differently on docker API.
Dind setup works great for Java and Node.

@mausch @jonesbusy I added support for mounts in hostconfig, that fixed ryuk support for dotnet at my end.

Nice! I'll give it a try in a few days, thanks!


How compile DLL with my code?

If i write code where calling MessageBox - error compile. How i can instead MessageBox my code, example: load file using curl. Help please!

error compile? show us the error ...

In ReflectiveDll.c wtite
#include "ReflectiveLoader.h"
#include "iostream"
using namespace std;

Error: required ";" in using namespace std; How?
http://prntscr.com/ezh4o8 Screen.

Because you are compiling for the C Language and not C++
ReflectiveDll.c should be ReflectiveDll.cpp
iostream is a template libarly - it's not available in C
http://stackoverflow.com/questions/14746425/are-there-templates-in-the-c-programming-language
using namespace std;
http://stackoverflow.com/questions/4396140/why-doesnt-ansi-c-have-namespaces
Think you should learn the language first before attempting this form of module loading.
GitHub if not for Tutorials in Programming.

Wow, I did not notice this. Thx!


Uncaught TypeError: Cannot read property 'id' of null

[Enter steps to reproduce:]

...
...

Atom: 1.19.0 x64
Electron: 1.6.9
OS: Mac OS X 10.12.6
Thrown From: Atom Core
Stack Trace
Uncaught TypeError: Cannot read property 'id' of null
At /Applications/Atom.app/Contents/Resources/app/src/text-editor-component.js:2168

TypeError: Cannot read property 'id' of null
    at horizontalPositionsToMeasure.forEach (/Applications/Atom.app/Contents/Resources/app/src/text-editor-component.js:2168:73)
    at Map.forEach (native)
    at TextEditorComponent.measureHorizontalPositions (/Applications/Atom.app/Contents/Resources/app/src/text-editor-component.js:2164:45)
    at TextEditorComponent.measureContentDuringUpdateSync (/Applications/Atom.app/Contents/Resources/app/src/text-editor-component.js:385:16)
    at scheduler.readDocument (/Applications/Atom.app/Contents/Resources/app/src/text-editor-component.js:270:20)
    at ViewRegistry.module.exports.ViewRegistry.performDocumentUpdate (/Applications/Atom.app/Contents/Resources/app/src/view-registry.js:191:15)
    at /Applications/Atom.app/Contents/Resources/app/src/view-registry.js:3:65

Commands
     -1:27.5.0 snippets:next-tab-stop (input.hidden-input)
     -1:27.5.0 snippets:expand (input.hidden-input)
     -1:27.5.0 editor:indent (input.hidden-input)
     -1:26.5.0 snippets:next-tab-stop (input.hidden-input)
     -1:26.5.0 snippets:expand (input.hidden-input)
     -1:26.5.0 editor:indent (input.hidden-input)
  7x -1:21.5.0 core:backspace (input.hidden-input)
  3x -1:16 core:move-right (input.hidden-input)
  2x -1:12.9.0 core:backspace (input.hidden-input)
     -1:10.9.0 core:save (input.hidden-input)
     -1:09.8.0 symbols-view:toggle-file-symbols (input.hidden-input)
     -0:54.7.0 core:backspace (input.hidden-input)
  3x -0:53.4.0 core:save (input.hidden-input)
     -0:42.2.0 find-and-replace:select-next (input.hidden-input)
     -0:39.3.0 find-and-replace:show (input.hidden-input)
 16x -0:38.2.0 core:confirm (input.hidden-input)

Non-Core Packages
atom-beautify 0.30.4
atom-material-syntax 1.0.6
atom-material-ui 2.0.4
atom-monokai 0.10.9
busy-signal 1.4.3
file-icons 2.1.10
intentions 1.1.5
jshint 1.8.6
linter 2.2.0
linter-eslint 8.2.1
linter-rubocop 2.1.1
linter-ui-default 1.6.3
monokai-seti 0.9.0
pigments 0.40.1
react 0.16.2
rspec 0.3.4
scope-inspector 1.0.1
seti-ui 1.9.0

Thanks for taking the time to contribute!
We noticed that this is a duplicate of https://github.com/atom/atom/issues/15168. You may want to subscribe there for updates - if you have consistent steps to reproduce in safe mode, please let us know in the issue.
Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


Create a caption-cleaning and -importing process

[x] create a method to reduce rolling captions to straight captions
[x] create a method to time-shift captions, so they're not ~10 seconds behind
[x] create a method to store captions in the video table
[x] create a method to store transcripts in the video table
[ ] create a method to identify the speaker of each caption, using video_index data
[x] create a utility to process a given SRT file

Identifying the speaker of each caption is probably best done with video_clips, which has already done the work of figuring out when each legislator starts and stops speaking.

In time-shifting captions, we're dropping microseconds. But those turn out to be useful to keep on hand. Modify that functionality to retain microseconds.

Microseconds are retained now.

Hey, here's a good way to identify speakers: by the fact that they are generally introduced.
So, here are a few rules:

Look at the last few lines before a speaker shift, and see if "SENATOR X" or "DELEGATE X" is referred to. That will probably be the next speaker.
When identifying a speaker, see if they say "Mr. President" or "Mr. Speaker." (Allow wiggle-room for gender, past be damned.) If so, then we know that it's not the House Speaker or the Senate President speaking. House Rule 54 actually requires that they say "Mr. Speaker."
If the speaker says things like "the clerk will close the roll," "are the senators ready to vote,"

I'm about 1/3 of the way done with this. Maybe closer to 2/3—tough to say. Anyway, this looks pretty straightforward.

Done. Works good enough!


Conformance: UTxO plumbing and translation

Description
Resolves #4256
Checklist

[ ] Commit sequence broadly makes sense and commits have useful messages
[ ] New tests are added if needed and existing tests are updated
[ ] When applicable, versions are updated in .cabal and CHANGELOG.md files according to the
versioning process.
[ ] The version bounds in .cabal files for all affected packages are updated. If you change the bounds in a cabal file, that package itself must have a version increase. (See RELEASING.md)
[ ] All visible changes are prepended to the latest section of a CHANGELOG.md for the affected packages. New section is never added with the code changes. (See RELEASING.md)
[x] Code is formatted with fourmolu (use scripts/fourmolize.sh)
[x] Cabal files are formatted (use scripts/cabal-format.sh)
[x] hie.yaml has been updated (use scripts/gen-hie.sh)
[ ] Self-reviewed the diff

Hey @aniketd, is this PR still necessary? I suspect @Soupstraw has his own version already implemeted, rgiht?

Closing in favor of #4660


Bump Bottlerocket release version to latest

Backporting #2879 to release-0.18 branch.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/hold

/unhold
/approve
/lgtm


⚠ ClamAV has degraded performance

In a71ec58, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 442 ms

Resolved: ClamAV performance has improved in 7dd023a.


knapsack: break when capacity is 0.0

Avoid the additional iterations once we are done.
Signed-off-by: Raul Gutierrez Segales<EMAIL_ADDRESS>

Thanks for the contribution. It's a good addition if this were more than an educational example, but the bar for making the source code in this repository diverge from the source code in the printed book which can't be changed is very high. I'm currently only accepting bug fixes, not performance improvements. Thanks.


An unhandled exception of type 'System.AccessViolationException' occurred in Grib.Api.dll

I Find a special case when this exception is thrown
in my code i have
function1()
{
try
{
using (GribFile gribFile = new GribFile(readPath))
{
//some work
gribFile.Dispose();
GC.Collect();
GC.WaitForPendingFinalizers();
}
}
catch (Exception ex)
{
GC.Collect();
GC.WaitForPendingFinalizers();
}
}
function2()
{
XDocument doc=XDocument.Parse(bigxml);
}
void main()
{
function1();
function2();
}
the above main is executed and works perfectly when we have complete file,both functions are executed properly. but when readPath Files is empty(0KB), the execution goes to catch statement with error This file is empty or invalid.
and  function1 completes.when function2 is called and it works with big data, it is throwing
An unhandled exception of type 'System.AccessViolationException' occurred in Grib.Api.dll
Additional information: Attempted to read or write protected memory. This is often an indication that other memory is corrupt.
function2 has not any call to gribFile
if you can replicate this case please resolve this issue,
i am using V6.0.3,
sounds like jump to catch does not dispose gribFile completly
Thank you

Ok, I'll take a look at it. Do you need a temporary workaround? Should be simple.
You shouldn't need to make any of those calls to GC. Dispose gets called when control leaves the using block. That in turn calls GC::SuppressFinalize. Furthermore, Collect only collects managed memory.

For now, you can use FileInfo::Length to check the file length before creating the GribFile. https://msdn.microsoft.com/en-us/library/system.io.fileinfo.length(v=vs.110).aspx

Thanks for quick resolution, i will implement it and reply .
but this is  not complete as reason of going to cath may be other
like Half downloaded file will throw error of end of file reached
Still my temp solution is done thank you

and also dispose and collect was used just for purpose of safety nothing else

do we have any progress in resolution of this bug?

Not yet. Is the workaround working for you?
On Dec 30, 2015 5:54 AM, "VishmayShah"<EMAIL_ADDRESS>wrote:

do we have any progress in resolution of this bug?
—
Reply to this email directly or view it on GitHub
https://github.com/0x1mason/GribApi.NET/issues/18#issuecomment-167976456
.

Half downloaded file will throw error of end of file reached.
so not 100% working

Ok, sorry about that. I'll try to get to it by Monday. Until then, try the following:

Download a hex editor, eg HxD
Open a complete file and check the very last characters. They're probably "7777". If not, check the spec for your GRIB version and find out the terminating characters for your GRIB message.
In addition to the FileInfo::Length check, read the file and check for the terminating characters.

Hi Vishmay,
Thanks for your patience. Can you try installing 0.6.4-beta and let me know if it resolves your issues?
You should get a FileLoadException for the empty file and a GribApiException for an incomplete file. You'll have to handle those exceptions as you would other valid exceptions. However, you shouldn't get any AccessViolationExceptions.

@VishmayShah I'm hoping this is resolved for you. If not, let me know.

For the actual 0.6.4 release, I've changed it so both situations throw FileLoadException. I also hardened the EOF check for 7777.

i am not sure on will it work or not as i am not upgrading it now as it is used.
the origin of problem was that in try catch block if flow goes to catch block then gribapi was not disposing
so we think that not let flow go in catch, so that check for empty file if not then try to read else don’t initialize grib api
same was needed for half downloaded files (that is pending in my proj)
so FileLoadException will properly dispose grib.api for 7777 is required. is it solving it? and how to use it?
From: Eric Millin
Sent: Sunday, January 24, 2016 5:10 AM
To: 0x1mason/GribApi.NET
Cc: VishmayShah
Subject: Re: [GribApi.NET] An unhandled exception of type 'System.AccessViolationException' occurred in Grib.Api.dll (#18)
For the actual 0.6.4 release, I've changed it so both situations throw FileLoadException. I also hardened the EOF check for 7777.
—
Reply to this email directly or view it on GitHub.

Thank you for your great responce

If your current solution works for you, then I would not make any changes. Here's how you would use the new changes:
SomeType data = null;
try
{
  using (GribFile file = new GribFile("mygrib.grb"))
  {
    // set `data`
  }
} catch (FileLoadException e)
{
  Console.WriteLine("The file is empty or incomplete: " + e.Message);
}

if (data == null)
{
  // handle failure
}


V6 - Checkbox as a custom element

I'm trying to do a custom checkbox component, but it's a bit weird. Simple example:
<Field
  name="isVisible"
  component={ props => <input type="checkbox" {...props} /> }
/>

This component will have a value prop, but not the checked prop. Even initial boolean value doesn't help. So in order to make it work, we need to do so:
<Field
  name="isVisible"
  component={ props => <input type="checkbox" {...props} /> }
  type="checkbox"
/>

Is it supposed to work this way?

@VladShcherbin I'd say that's working as intended, Field needs to know what kind of component it's working with

Yes, that is working as expected. The recommended way to do that is:
<Field name="isVisible" component="input" type="checkbox"/>

@babsonmatt @erikras I still don't get it. This was just a simple example. A little bit bigger one:
const Checkbox = props => (

    {props.label}
    <input type="checkbox" id={props.name} {...props}
    {props.touched && props.error && (

{props.error}
    )}

)

Now, we can use it this way:
<Field name="isVisible" component={Checkbox} />

I expect the custom Checkbox component to be complete, so I can use the above syntax. But, unfortunately, I will always need to add type to it so it looks like this:
<Field name="isVisible" component={Checkbox} type="checkbox" />

For me, this is weird as I expect custom component to be independent. Maybe it is possible to make Field understand, when to add checked prop (for example from type of initial value)
Even in the docs:

checked
An alias for value only when value is a boolean. Provided for convenience of destructuring the whole field object into the props of a form element.

The initial value is boolean, but still it has value, but not the checked prop.


Azure Standard_B1s VM incorrectly priced as $0

infracost breakdown --path . --terraform-parse-hcl
 azurerm_linux_virtual_machine.redacted1
 ├─ Instance usage (pay as you go, Standard_B2s)                 730  hours                           $35.04
 └─ os_disk
    ├─ Storage (E6)                                                1  months                           $4.80
    └─ Disk operations                            Monthly cost depends on usage: $0.002 per 10k operations

 azurerm_linux_virtual_machine.redacted2
 ├─ Instance usage (pay as you go, Standard_B1s)                 730  hours                            $0.00
 └─ os_disk
    ├─ Storage (E6)                                                1  months                           $4.80
    └─ Disk operations                            Monthly cost depends on usage: $0.002 per 10k operations

Running with --log-level trace gives some relevant logging:
time="redacted" level=debug msg="Multiple products found for azurerm_linux_virtual_machine.redacted2 Instance usage (pay as you go, Standard_B1s), filtering those with prices"
time="redacted" level=warning msg="Multiple products with prices found for azurerm_linux_virtual_machine.redacted2 Instance usage (pay as you go, Standard_B1s), using the first product"

This logging is not shown for the Standard_B2s VM.

Thanks for the issue, we'll check it out!

@tvdstaaij The fix for this will go live in the next release.


UPDATE: jupyter-cache v0.6.0

Bumping jupyter cache versions

Merging this since this is a no-op right now, and we'll release a new jupyter-cache soon


K-bert cannot achieve performance as paper mention

I use k-bert run_ner.py but cannot get the result as high as the paper say, and I dont know how to change dataset to finetune

We provide a third-party implementation of K-BERT. If there is a problem, please consult the original authors of the paper.


Getting started

Hello,
I have just started to use your code. Unfortunately with limited experience in NodeJS.
My attempt to use the test script to output telemetry data to the console fails.
I've installed NodeJS 18.6 and NPM 8.13.2 on a Debian 11, installed with 'npm install f1-22-udp'. So i tried to start the example (node example.js) :
import { F122UDP } from "f1-22-udp";
/*
*   'port' is optional, defaults to 20777
*   'address' is optional, defaults to localhost
*/

const f122: F122UDP = new F122UDP();
f122.start();
// motion 0
client.on('motion',function(data) {
    console.log(data);
})

but getting the following error:
file:///opt/f1-22-udp/motion.js:7
const f122: F122UDP = new F122UDP();
      ^^^^

SyntaxError: Missing initializer in const declaration
    at ESMLoader.moduleStrategy (node:internal/modules/esm/translators:119:18)
    at ESMLoader.moduleProvider (node:internal/modules/esm/loader:483:14)
    at async link (node:internal/modules/esm/module_job:67:21)

Node.js v18.6.0

Thanks in advance.

the example script is a bit wrong, it should be
import { F122UDP } from "f1-22-udp";
/*
*   'port' is optional, defaults to 20777
*   'address' is optional, defaults to localhost
*/

const f122 = new F122UDP()
f122.start();
// motion 0
f122.on('motion', function (data) {
    console.log(data);
})

f122.on("event", data => {
    console.log(data);
})

the example code is in typescript, removing the type annotation will fix the problem.
const f122: F122UDP = new F122UDP();

change to

const f122 = new F122UDP();

Thank you guys!


net: tcp: Bring essential TCP fixes from upstream

It was requested by the Community to incorporate latest TCP fixes into NCS 2.0 release.

Dear reviewers, I know it's not your area, but I still need 2 approvals to get this in, and there's not really anyone beside me that could review this. The commits are just cherry-picks from upstream, requested by upstream NCS users.

@AndreyDodonov-EH FYI, the commit list should be complete, but please let me know if I missed something (I did not include tests in the list on purpose).

@AndreyDodonov-EH FYI, the commit list should be complete, but please let me know if I missed something (I did not include tests in the list on purpose).

@rlubos

I think you also fixed packet leak in net_shell.c ?
I find your DNS fix also important: https://github.com/zephyrproject-rtos/zephyr/pull/42321
(though that's not strictly TCP, but I think it makes sense to integrate it nevertheless)
That's not TCP at all, but I found this fix quite critical: https://github.com/zephyrproject-rtos/zephyr/pull/44090, would be happy if it also lands in 2.0 and in Zephyr. It was causing spontaneous reboots when logging a lot, which was hard to track.
May be makes sense to merge it and backport it in a separate PR.

Packet leak fix in net_shell was brought with the recent upmerge.

May be makes sense to merge it and backport it in a separate PR.

Yes, let's focus on networking stuff in this PR.

@rlubos
I'd like to share intermediate test results.
I checked out complete networking (sybsys/net and include) from your branch.
Everything seems fine, except for downloading - sometimes when downloading big files I just receive less data on HTTP level.
(E.g. requesting 1 MB and receiving 800 KB or so).
I'll try to reproduce it based on big_http_download sample on QEMU and not just on target (where there is some packet loss definitely, it's medium-quality Wi-Fi).
In terms of configuration, I don't set RX window explicitly or buffer count (so it remains default 36).
Once I have a tangible example to reproduce, I'll share it here.


Read config from remote config server

Signed-off-by: Matto<EMAIL_ADDRESS>Hey, I just made a Pull Request!

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Thank you! 👍
Overall I think we can work this in but I'd like to minimize the impact for anyone that doesn't need the feature

Would you prefer if I create it in a new class maybe RemoteLoader, RemoteObservableConfigProxy. This way anyone who is not interested in it would not use it in the first place?
The way we are running it, we building a dashboard for updating the the backend and the frontend without restarts or builds. So it's important for us to be able to read from config from a url. Next step for me was to work on refreshing the front end on config update. Does this violates your sterategy?

@Rugvip Any chance we get this release this week before it gets any conflict.

@Rugvip Reran prettier on all the files, can you please trigger the checks again?

@Rugvip is there any way I can run these checks without asking you for review every time? just mindful of your time.
I re committed the api-report.md

I don't know of a way to do that, for the very first time contributors.  I triggered it now though

I don't know of a way to do that, for the very first time contributors.  I triggered it now though

@freben Great! thank you. Do you think it can be merged and released this week?

Regular releases are on Thursdays. This particular week it was skipped though, since the maintainers are all off.